The following checklist is intended to help maintainers decide on whether to merge, request changes or close a PR. It also brings more transparency for contributors in addition to Acceptable Formulae requirements.
This is a guiding principle. As a maintainer, you can make a call to either request changes from a contributor or help them out based on their comfort and previous contributions. Remember, as a team we Prioritise Maintainers Over Users to avoid burnout.
This is a more practical checklist; it should be used after you get familiar with Maintainer Guidelines.
inreplacethat have been applied to upstream and can be removed
url, as we do skip some versions (for example vim or v8)
foobar: fix flibble matrix
bottle block is not removed
Please keep bottle block in place, [@BrewTestBot](https://github.com/BrewTestBot) takes care of it.
# in this example PR is for `libuv` formula and `urbit` needs revision bump brew bump-revision --message 'for libuv' urbit
bottle :unneeded, you can merge it through GitHub UI
automerge-skip): approve the PR to trigger an automatic merge (use
brew pr-publish $PR_IDto trigger manually in case of a new formula)
brew pr-publish $PR_ID
brew pr-pull $PR_ID, make changes, and
brew bump-formula-prnext time if this was not the case
© 2009–present Homebrew contributors
Licensed under the BSD 2-Clause License.