brew/docs/How-To-Open-a-Homebrew-Pull-Request.md
Ryan Schumacher ff6df8ee56
Update How-To-Open-a-Homebrew-Pull-Request.md
Co-authored-by: Carlo Cabrera <30379873+carlocab@users.noreply.github.com>
2024-08-29 18:45:10 -05:00

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

  1. Use brew bump-formula-pr to do everything (i.e. forking, committing, pushing) with a single command. Run brew bump-formula-pr --help to learn more.

Submit a new version of an existing cask

  1. Use brew bump-cask-pr to do everything (i.e. forking, committing, pushing) with a single command. Run brew bump-cask-pr --help to learn more.

Set up your own fork of the Homebrew repository

Core brew code pull request

  1. 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.
  2. Change to the directory containing your Homebrew installation:

    cd "$(brew --repository)"
    
  3. 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.

Before creating a new formulae, please read Acceptable Formulae.

  1. 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.
  2. Tap (download a local clone of) the repository of core Homebrew formulae:

    brew tap --force homebrew/core
    
  3. Change to the directory containing Homebrew formulae:

    cd "$(brew --repository homebrew/core)"
    
  4. 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.

Before creating a new cask, please read Acceptable Casks.

  1. 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.
  2. Tap (download a local clone of) the repository of core Homebrew casks:

    brew tap --force homebrew/cask
    
  3. Change to the directory containing Homebrew casks:

    cd "$(brew --repository homebrew/cask)"
    
  4. 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:

  1. Check out the master branch:

    git checkout master
    
  2. Retrieve new changes to the master branch:

    brew update
    
  3. Create a new branch from the latest master branch:

    git checkout -b <YOUR_BRANCH_NAME> origin/master
    
  4. 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.
  5. 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>
    
  6. Make a separate commit for each changed formula with git add and git 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".
  7. Upload your branch of new commits to your fork:

    git push --set-upstream <YOUR_USERNAME> <YOUR_BRANCH_NAME>
    
  8. 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.

  9. 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:

  1. Ask for clarification of anything you don't understand and for help with anything you don't know how to do.
  2. 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.
  3. Keep discussion in the pull request unless requested otherwise (i.e. do not email maintainers privately).
  4. Do not continue discussion in closed pull requests.
  5. 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:

  1. Check out your branch again:

    git checkout <YOUR_BRANCH_NAME>
    
  2. Make any requested changes and commit them with git add and git commit.

  3. 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.
  4. 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!