2015-03-12 23:29:10 +00:00
|
|
|
page_title: Participate in the PR Review
|
|
|
|
page_description: Basic workflow for Docker contributions
|
2015-03-21 22:30:29 +00:00
|
|
|
page_keywords: contribute, pull request, review, workflow, beginner, squash, commit
|
2015-03-12 23:29:10 +00:00
|
|
|
|
|
|
|
|
|
|
|
# Participate in the PR Review
|
|
|
|
|
|
|
|
Creating a pull request is nearly the end of the contribution process. At this
|
|
|
|
point, your code is reviewed both by our continuous integration (CI) systems and
|
|
|
|
by our maintainers.
|
|
|
|
|
|
|
|
The CI system is an automated system. The maintainers are human beings that also
|
|
|
|
work on Docker. You need to understand and work with both the "bots" and the
|
|
|
|
"beings" to review your contribution.
|
|
|
|
|
|
|
|
|
|
|
|
## How we proces your review
|
|
|
|
|
|
|
|
First to review your pull request is Gordon. Gordon is fast. He checks your
|
|
|
|
pull request (PR) for common problems like a missing signature. If Gordon finds a
|
|
|
|
problem, he'll send an email through your GitHub user account:
|
|
|
|
|
|
|
|
![Gordon](/project/images/gordon.jpeg)
|
|
|
|
|
|
|
|
Our build bot system starts building your changes while Gordon sends any emails.
|
|
|
|
|
|
|
|
The build system double-checks your work by compiling your code with Docker's master
|
|
|
|
code. Building includes running the same tests you ran locally. If you forgot
|
|
|
|
to run tests or missed something in fixing problems, the automated build is our
|
|
|
|
safety check.
|
|
|
|
|
|
|
|
After Gordon and the bots, the "beings" review your work. Docker maintainers look
|
|
|
|
at your pull request and comment on it. The shortest comment you might see is
|
|
|
|
`LGTM` which means **l**ooks-**g**ood-**t**o-**m**e. If you get an `LGTM`, that
|
|
|
|
is a good thing, you passed that review.
|
|
|
|
|
|
|
|
For complex changes, maintainers may ask you questions or ask you to change
|
|
|
|
something about your submission. All maintainer comments on a PR go to the
|
|
|
|
email address associated with your GitHub account. Any GitHub user who
|
|
|
|
"participates" in a PR receives an email to. Participating means creating or
|
|
|
|
commenting on a PR.
|
|
|
|
|
|
|
|
Our maintainers are very experienced Docker users and open source contributors.
|
|
|
|
So, they value your time and will try to work efficiently with you by keeping
|
|
|
|
their comments specific and brief. If they ask you to make a change, you'll
|
|
|
|
need to update your pull request with additional changes.
|
|
|
|
|
|
|
|
## Update an Existing Pull Request
|
|
|
|
|
|
|
|
To update your existing pull request:
|
|
|
|
|
|
|
|
1. Change one or more files in your local `docker-fork` repository.
|
|
|
|
|
|
|
|
2. Commit the change with the `git commit --amend` command.
|
|
|
|
|
|
|
|
$ git commit --amend
|
|
|
|
|
|
|
|
Git opens an editor containing your last commit message.
|
|
|
|
|
|
|
|
3. Adjust your last comment to reflect this new change.
|
|
|
|
|
|
|
|
Added a new sentence per Anaud's suggestion
|
|
|
|
|
|
|
|
Signed-off-by: Mary Anthony <mary@docker.com>
|
|
|
|
|
|
|
|
# Please enter the commit message for your changes. Lines starting
|
|
|
|
# with '#' will be ignored, and an empty message aborts the commit.
|
|
|
|
# On branch 11038-fix-rhel-link
|
|
|
|
# Your branch is up-to-date with 'origin/11038-fix-rhel-link'.
|
|
|
|
#
|
|
|
|
# Changes to be committed:
|
|
|
|
# modified: docs/sources/installation/mac.md
|
|
|
|
# modified: docs/sources/installation/rhel.md
|
|
|
|
|
|
|
|
4. Push to your origin.
|
|
|
|
|
|
|
|
$ git push origin
|
|
|
|
|
|
|
|
5. Open your browser to your pull request on GitHub.
|
|
|
|
|
|
|
|
You should see your pull request now contains your newly pushed code.
|
|
|
|
|
|
|
|
6. Add a comment to your pull request.
|
|
|
|
|
|
|
|
GitHub only notifies PR participants when you comment. For example, you can
|
|
|
|
mention that you updated your PR. Your comment alerts the maintainers that
|
|
|
|
you made an update.
|
|
|
|
|
|
|
|
A change requires LGTMs from an absolute majority of an affected component's
|
|
|
|
maintainers. For example, if you change `docs/` and `registry/` code, an
|
|
|
|
absolute majority of the `docs/` and the `registry/` maintainers must approve
|
|
|
|
your PR. Once you get approval, we merge your pull request into Docker's
|
|
|
|
`master` code branch.
|
|
|
|
|
|
|
|
## After the merge
|
|
|
|
|
|
|
|
It can take time to see a merged pull request in Docker's official release.
|
|
|
|
A master build is available almost immediately though. Docker builds and
|
|
|
|
updates its development binaries after each merge to `master`.
|
|
|
|
|
|
|
|
1. Browse to <a href="https://master.dockerproject.com/" target="_blank">https://master.dockerproject.com/</a>.
|
|
|
|
|
|
|
|
2. Look for the binary appropriate to your system.
|
|
|
|
|
|
|
|
3. Download and run the binary.
|
|
|
|
|
|
|
|
You might want to run the binary in a container though. This
|
|
|
|
will keep your local host environment clean.
|
|
|
|
|
|
|
|
4. View any documentation changes at <a href="http://docs.master.dockerproject.com/" target="_blank">docs.master.dockerproject.com</a>.
|
|
|
|
|
|
|
|
Once you've verified everything merged, feel free to delete your feature branch
|
|
|
|
from your fork. For information on how to do this,
|
|
|
|
<a href="https://help.github.com/articles/deleting-unused-branches/" target="_blank">
|
|
|
|
see the GitHub help on deleting branches</a>.
|
|
|
|
|
|
|
|
## Where to go next
|
|
|
|
|
|
|
|
At this point, you have completed all the basic tasks in our contributors guide.
|
2015-03-21 22:30:29 +00:00
|
|
|
If you enjoyed contributing, let us know by completing another beginner
|
2015-03-12 23:29:10 +00:00
|
|
|
issue or two. We really appreciate the help.
|
|
|
|
|
|
|
|
If you are very experienced and want to make a major change, go on to
|
|
|
|
[learn about advanced contributing](/project/advanced-contributing).
|