2015-03-12 19:29:10 -04:00
|
|
|
page_title: Create a pull request (PR)
|
|
|
|
page_description: Basic workflow for Docker contributions
|
2015-03-21 18:30:29 -04:00
|
|
|
page_keywords: contribute, pull request, review, workflow, beginner, squash, commit
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
# Create a pull request (PR)
|
|
|
|
|
|
|
|
A pull request (PR) sends your changes to the Docker maintainers for review. You
|
|
|
|
create a pull request on GitHub. A pull request "pulls" changes from your forked
|
|
|
|
repository into the `docker/docker` repository.
|
|
|
|
|
|
|
|
You can see <a href="https://github.com/docker/docker/pulls" target="_blank">the
|
|
|
|
list of active pull requests to Docker</a> on GitHub.
|
|
|
|
|
2015-04-21 11:50:09 -04:00
|
|
|
## Check your work
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
Before you create a pull request, check your work.
|
|
|
|
|
|
|
|
1. In a terminal window, go to the root of your `docker-fork` repository.
|
|
|
|
|
|
|
|
$ cd ~/repos/docker-fork
|
|
|
|
|
|
|
|
2. Checkout your feature branch.
|
|
|
|
|
|
|
|
$ git checkout 11038-fix-rhel-link
|
2015-04-15 17:11:58 -04:00
|
|
|
Switched to branch '11038-fix-rhel-link'
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
3. Run the full test suite on your branch.
|
|
|
|
|
|
|
|
$ make test
|
|
|
|
|
|
|
|
All the tests should pass. If they don't, find out why and correct the
|
|
|
|
situation.
|
|
|
|
|
|
|
|
4. Optionally, if modified the documentation, build the documentation:
|
|
|
|
|
|
|
|
$ make docs
|
|
|
|
|
|
|
|
5. Commit and push any changes that result from your checks.
|
|
|
|
|
|
|
|
## Rebase your branch
|
|
|
|
|
|
|
|
Always rebase and squash your commits before making a pull request.
|
|
|
|
|
2015-04-15 17:11:58 -04:00
|
|
|
1. Checkout your feature branch in your local `docker-fork` repository.
|
|
|
|
|
|
|
|
This is the branch associated with your request.
|
|
|
|
|
|
|
|
2. Fetch any last minute changes from `docker/docker`.
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
$ git fetch upstream master
|
|
|
|
From github.com:docker/docker
|
|
|
|
* branch master -> FETCH_HEAD
|
|
|
|
|
|
|
|
3. Start an interactive rebase.
|
|
|
|
|
|
|
|
$ git rebase -i upstream/master
|
|
|
|
|
|
|
|
4. Rebase opens an editor with a list of commits.
|
|
|
|
|
|
|
|
pick 1a79f55 Tweak some of the other text for grammar
|
|
|
|
pick 53e4983 Fix a link
|
|
|
|
pick 3ce07bb Add a new line about RHEL
|
|
|
|
|
2015-04-15 17:11:58 -04:00
|
|
|
5. Replace the `pick` keyword with `squash` on all but the first commit.
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
pick 1a79f55 Tweak some of the other text for grammar
|
|
|
|
squash 53e4983 Fix a link
|
|
|
|
squash 3ce07bb Add a new line about RHEL
|
|
|
|
|
2015-04-15 17:11:58 -04:00
|
|
|
After you save the changes and quit from the editor, git starts
|
|
|
|
the rebase, reporting the progress along the way. Sometimes
|
|
|
|
your changes can conflict with the work of others. If git
|
|
|
|
encounters a conflict, it stops the rebase, and prints guidance
|
|
|
|
for how to correct the conflict.
|
2015-03-12 19:29:10 -04:00
|
|
|
|
2015-04-15 17:11:58 -04:00
|
|
|
6. Edit and save your commit message.
|
2015-03-12 19:29:10 -04:00
|
|
|
|
2015-03-15 00:28:55 -04:00
|
|
|
`git commit -s`
|
|
|
|
|
2015-04-25 14:57:01 -04:00
|
|
|
Make sure your message includes <a href="./set-up-git" target="_blank>your signature</a>.
|
2015-03-12 19:29:10 -04:00
|
|
|
|
2015-04-15 17:11:58 -04:00
|
|
|
7. Force push any changes to your fork on GitHub.
|
2015-03-12 19:29:10 -04:00
|
|
|
|
2015-04-15 17:11:58 -04:00
|
|
|
$ git push -f origin 11038-fix-rhel-link
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
## Create a PR on GitHub
|
|
|
|
|
|
|
|
You create and manage PRs on GitHub:
|
|
|
|
|
|
|
|
1. Open your browser to your fork on GitHub.
|
|
|
|
|
|
|
|
You should see the latest activity from your branch.
|
|
|
|
|
|
|
|
![Latest commits](/project/images/latest_commits.png)
|
|
|
|
|
|
|
|
|
|
|
|
2. Click "Compare & pull request."
|
|
|
|
|
|
|
|
The system displays the pull request dialog.
|
|
|
|
|
|
|
|
![PR dialog](/project/images/to_from_pr.png)
|
|
|
|
|
|
|
|
The pull request compares your changes to the `master` branch on the
|
|
|
|
`docker/docker` repository.
|
|
|
|
|
|
|
|
3. Edit the dialog's description and add a reference to the issue you are fixing.
|
|
|
|
|
|
|
|
GitHub helps you out by searching for the issue as you type.
|
|
|
|
|
|
|
|
![Fixes issue](/project/images/fixes_num.png)
|
|
|
|
|
|
|
|
4. Scroll down and verify the PR contains the commits and changes you expect.
|
|
|
|
|
|
|
|
For example, is the file count correct? Are the changes in the files what
|
2015-04-15 17:11:58 -04:00
|
|
|
you expect?
|
2015-03-12 19:29:10 -04:00
|
|
|
|
|
|
|
![Commits](/project/images/commits_expected.png)
|
|
|
|
|
|
|
|
5. Press "Create pull request".
|
|
|
|
|
|
|
|
The system creates the request and opens it for you in the `docker/docker`
|
|
|
|
repository.
|
|
|
|
|
|
|
|
![Pull request made](/project/images/pull_request_made.png)
|
|
|
|
|
|
|
|
|
|
|
|
## Where to go next
|
|
|
|
|
|
|
|
Congratulations, you've created your first pull request to Docker. The next
|
|
|
|
step is for you learn how to [participate in your PR's
|
2015-04-15 17:11:58 -04:00
|
|
|
review](/project/review-pr/).
|