-
Notifications
You must be signed in to change notification settings - Fork 0
CodeReview
Be sure to familiarize yourself with the code review process from the official Contribution Guidelines first.
There are several terms code reviews may use that you should become familiar with.
-
LGTM
— looks good to me -
SGTM
— sounds good to me -
s/foo/bar/
— please replacefoo
withbar
; this is sed syntax. -
s/foo/bar/g
— please replacefoo
withbar
throughout your entire change
Messages from a code review are typically sent to three places:
- the reviewers, if any
- the golang-codereviews group
- the owner
Please do NOT reply code review via email, as the message will not be relayed to Gerrit. Always click on the link and post reply in Gerrit.
If you have changes that aren't ready to be reviewed, you can put a giant "DO NOT REVIEW" as the second line of the CL description, so that people who see it know not to look any further. Don't make it the first line, since then it will become the subject for the entire review, even after you've changed the description.
If you don't need Gerrit's features, but just want to backup your work, share work between multiple clients, or have a staging UI to examine your changes, you can use a regular git remote.
To use GitHub as a git remote, you can either fork github.com/golang/go or create a new repo. There are trade-offs. Forked repos will have a faster first push. Non-forked repos can be private. Forked repos are associated in GitHub's system. As a result, they are easily discoverable and support cross-repo comparisons in the GitHub UI; however, this also means that references to issues in commit messages in forked repos will create references to your fork in the issue.
To add your git remote, run something like:
$ git remote add fork git@github.com:yourusername/go.git
You can then push changes to the "fork" remote with git push fork branchname
.
Gerrit's code review model is to rewrite a single commit until it is correct. GitHub will try to prevent you from accidentally overwriting your existing branch. You can work around this by forcing the push: git push --force fork branchname
. Alternatively, you can set up your forked remote as a mirror by cloning it initially with:
$ git remote add --mirror=push fork git@github.com:yourusername/go.git
Then running git push fork
will update GitHub to perfectly mirror everything (all branches, all tags, etc.). This is handy, but take care when using this on multiple clients. You are bypassing the usual git safeguards, so it is easy to overwrite (and thus lose) work pushed by a different client.
- Home
- Getting started with Go
- Working with Go
- Learning more about Go
- The Go Community
- Using the Go toolchain
- Additional Go Programming Wikis
- Online Services that work with Go
- Troubleshooting Go Programs in Production
- Contributing to the Go Project
- Platform Specific Information
- Release Specific Information