Co-authored-by: Carlo Cabrera <30379873+carlocab@users.noreply.github.com>
7.7 KiB
How to Open a Homebrew Pull Request
The following commands are used by Homebrew contributors to set up a fork of Homebrew's Git repository on GitHub, create a new branch and create a GitHub pull request ("PR") for the changes in that branch.
The type of change you want to make influences which of Homebrew's main repositories you'll need to send your pull request to. If you want to submit a change to Homebrew's core code (the brew
implementation), you should open a pull request on Homebrew/brew. If you want to submit a change for a formula, you should open a pull request on the homebrew/core tap, while for casks you should open the pull request on the homebrew/cask tap or another official tap, depending on the formula type.
Submit a new version of an existing formula
- Use
brew bump-formula-pr
to do everything (i.e. forking, committing, pushing) with a single command. Runbrew bump-formula-pr --help
to learn more.
Submit a new version of an existing cask
- Use
brew bump-cask-pr
to do everything (i.e. forking, committing, pushing) with a single command. Runbrew bump-cask-pr --help
to learn more.
Set up your own fork of the Homebrew repository
Core brew
code pull request
-
Fork the Homebrew/brew repository on GitHub.
- This creates a personal remote repository that you can push to. This is needed because only Homebrew maintainers have push access to the main repositories.
-
Change to the directory containing your Homebrew installation:
cd "$(brew --repository)"
-
Add your pushable forked repository as a new remote:
git remote add <YOUR_USERNAME> https://github.com/<YOUR_USERNAME>/brew.git
<YOUR_USERNAME>
is your GitHub username, not your local machine username.
Formulae-related pull request
Before creating a new formulae, please read Acceptable Formulae.
-
Fork the Homebrew/homebrew-core repository on GitHub.
- This creates a personal remote repository that you can push to. This is needed because only Homebrew maintainers have push access to the main repositories.
-
Tap (download a local clone of) the repository of core Homebrew formulae:
brew tap --force homebrew/core
-
Change to the directory containing Homebrew formulae:
cd "$(brew --repository homebrew/core)"
-
Add your pushable forked repository as a new remote:
git remote add <YOUR_USERNAME> https://github.com/<YOUR_USERNAME>/homebrew-core.git
<YOUR_USERNAME>
is your GitHub username, not your local machine username.
Cask-related pull request
Before creating a new cask, please read Acceptable Casks.
-
Fork the Homebrew/homebrew-cask repository on GitHub.
- This creates a personal remote repository that you can push to. This is needed because only Homebrew maintainers have push access to the main repositories.
-
Tap (download a local clone of) the repository of core Homebrew casks:
brew tap --force homebrew/cask
-
Change to the directory containing Homebrew casks:
cd "$(brew --repository homebrew/cask)"
-
Add your pushable forked repository as a new remote:
git remote add <YOUR_USERNAME> https://github.com/<YOUR_USERNAME>/homebrew-cask.git
<YOUR_USERNAME>
is your GitHub username, not your local machine username.
Create your pull request from a new branch
To make changes on a new branch and submit it for review, create a GitHub pull request with the following steps:
-
Check out the
master
branch:git checkout master
-
Retrieve new changes to the
master
branch:brew update
-
Create a new branch from the latest
master
branch:git checkout -b <YOUR_BRANCH_NAME> origin/master
-
Make your changes. For formulae or casks, use
brew edit
or your favourite text editor, following all the guidelines in the Formula Cookbook or Cask Cookbook.- If there's a
bottle do
block in the formula, don't remove or change it; we'll update it when we merge your PR.
- If there's a
-
Test your changes by running the following, and ensure they all pass without issue. For changed formulae and casks, make sure you do the
brew audit
step after your changed formula/cask has been installed.brew tests HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <CHANGED_FORMULA|CHANGED_CASK> brew test <CHANGED_FORMULA|CHANGED_CASK> brew audit --strict --online <CHANGED_FORMULA|CHANGED_CASK>
-
Make a separate commit for each changed formula with
git add
andgit commit
. Each formula's commits must be squashed.- Please note that our required commit message format for simple version updates is "
<FORMULA_NAME> <NEW_VERSION>
", e.g. "source-highlight 3.1.8
".
- Please note that our required commit message format for simple version updates is "
-
Upload your branch of new commits to your fork:
git push --set-upstream <YOUR_USERNAME> <YOUR_BRANCH_NAME>
-
Go to the relevant repository (e.g. https://github.com/Homebrew/brew, https://github.com/Homebrew/homebrew-core, etc.) and create a pull request to request review and merging of the commits from your pushed branch. Explain why the change is needed and, if fixing a bug, how to reproduce the bug. Make sure you have done each step in the checklist that appears in your new PR.
-
Await feedback or a merge from Homebrew's maintainers. We typically respond to all PRs within a couple days, but it may take up to a week, depending on the maintainers' workload.
Thank you!
Following up
To respond well to feedback:
- Ask for clarification of anything you don't understand and for help with anything you don't know how to do.
- Post a comment on your pull request if you've provided all the requested changes/information and it hasn't been merged after a week. Post a comment on your pull request if you're stuck and need help.
- A
needs response
label on a PR means that the Homebrew maintainers need you to respond to previous comments.
- A
- Keep discussion in the pull request unless requested otherwise (i.e. do not email maintainers privately).
- Do not continue discussion in closed pull requests.
- Do not argue with Homebrew maintainers. You may disagree but unless they change their mind, please implement what they request. Ultimately they control what is included in Homebrew, as they have to support any changes that are made.
To make changes based on feedback:
-
Check out your branch again:
git checkout <YOUR_BRANCH_NAME>
-
Make any requested changes and commit them with
git add
andgit commit
. -
Squash new commits into one commit per formula:
git rebase --interactive origin/master
- If you are working on a PR for a single formula,
git commit --amend
is a convenient way of keeping your commits squashed as you go.
- If you are working on a PR for a single formula,
-
Push to your remote fork's branch and the pull request:
git push --force
Once all feedback has been addressed and if it's a change we want to include (we include most changes), then we'll add your changes to Homebrew. Note that the PR status may show up as "Closed" instead of "Merged" because of the way we merge contributions. Don't worry: you will still get author credit in the actual merged commit.
Well done, you are now a Homebrew contributor!