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)
erlang@17) that can be upgraded
foobar: fix flibble matrix
brew pullin case the PR have a single commit but the wrong message
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 :unneededyou can merge it through GitHub UI
brew pull --bottle $PR_ID
brew bump-formula-prnext time if this was not the case