FAQ for contributorsEstimated reading time: 4 minutes
This section contains some frequently asked questions and tips for troubleshooting problems in your code contribution.
- How do I set my signature?
- How do I track changes from the docker repo upstream?
- How do I format my Go code?
- What is the pre-pull request checklist?
- How should I comment my code?
- How do I rebase my feature branch?
How do I set my signature?
Change to the root of your
$ cd docker-fork
user.namefor the repository.
$ git config --local user.name "FirstName LastName"
user.emailfor the repository.
$ git config --local user.email "email@example.com"
How do I track changes from the docker repo upstream?
Set your local repo to track changes upstream, on the
Change to the root of your Docker repository.
$ cd docker-fork
Add a remote called
upstreamthat points to
$ git remote add upstream https://github.com/moby/moby.git
How do I format my Go code?
gofmt -s -w filename.go on each changed file before committing your changes.
Most editors have plug-ins that do the formatting automatically.
What is the pre-pull request checklist?
Sync and cleanly rebase on top of Docker’s
master; do not mix multiple branches in the pull request.
Squash your commits into logical units of work using
git rebase -iand
git push -f.
If your code requires a change to tests or documentation, include code, test, and documentation changes in the same commit as your code; this ensures a revert would remove all traces of the feature or fix.
Reference each issue in your pull request description (
How should I comment my code?
The Go blog wrote about code comments, it is a single page explanation. A summary follows:
- Comments begin with two forward
- To document a type, variable, constant, function, or even a package, write a regular comment directly preceding the elements declaration, with no intervening blank line.
- Comments on package declarations should provide general package documentation.
- For packages that need large amounts of introductory documentation: the package comment is placed in its own file.
- Subsequent lines of text are considered part of the same paragraph; you must leave a blank line to separate paragraphs.
- Indent pre-formatted text relative to the surrounding comment text (see gob’s doc.go for an example).
- URLs are converted to HTML links; no special markup is necessary.
How do I rebase my feature branch?
Always rebase and squash your commits before making a pull request.
Fetch any of the last minute changes from
$ git fetch upstream master
Start an interactive rebase.
$ git rebase -i upstream/master
Rebase opens an editor with a list of commits.
pick 1a79f55 Tweak some of images pick 3ce07bb Add a new line
If you run into trouble,
git --rebase abortremoves any changes and gets you back to where you started.
squashon all but the first commit.
pick 1a79f55 Tweak some of images squash 3ce07bb Add a new line
After closing the file,
gitopens your editor again to edit the commit message.
Edit and save your commit message.
$ git commit -s
Make sure your message includes your signature.
Push any changes to your fork on GitHub, using the
-foption to force the previous change to be overwritten.
$ git push -f origin my-keen-feature
How do I update vendor package from upstream?
If you are not using the development container, download the vndr vendoring tool. The
vndrtool is included in the development container.
Edit the package version in
vendor.confto use the package you want to use, such as
vndr <package-name>. For example: