-
Notifications
You must be signed in to change notification settings - Fork 17.7k
GerritBot
GerritBot is a tool used for importing GitHub Pull Requests (PRs) into Gerrit for code review. It was created because the Go team does all its reviews in Gerrit, but we'd like to allow a more common workflow for contributing code via Github PRs.
- A user can upload a PR against any of our GitHub repos just as they would with any other GitHub project that accepts PRs
- The PR changes will then be imported by GerritBot and a message will be posted to the GitHub PR Issue containing a link to the Gerrit review
- All comments are handled within Gerrit. Any comments on the GitHub PR will be ignored
- The PR author can continue to upload commits to the branch used by the PR in order to address feedback from Gerrit
- Once the code is ready to be merged, a maintainer will submit the change on Gerrit and GerritBot will close the issue
- Similarly, if a change is closed or abandoned on Gerrit, the corresponding PR will be closed
Please file an issue and use the x/build/cmd/gerritbot:
prefix in the title.
This is due to an open bug with the way Gerrit handles acting as another user. Once that is fixed, the original author will also be the owner of the change.
You can upload as many commits as you like. GerritBot will handle squashing your commits into one change that Gerrit can handle.
It uses the title and description of the PR to construct the commit message for the Gerrit Change.
This is a limitation of the infrastructure that runs our Gerrit instances and is out of our control, plus you already need a Google account to sign our CLA, a requirement for us to accept your contribution in the first place.
Replies to comments in Gerrit are first saved as drafts and need to be published via the “Reply” button. This is to prevent multiple emails per review “session” and is similar to the pending review workflow in GitHub.
- If the feature/bug is non-trivial, please file an issue first
- The code is located at x/build/cmd/gerritbot (GitHub, Gerrit)