Explain why advanced contribution workflow have to be so complex Signed-off-by: Maxim Kulkin <mkulkin@mirantis.com>
5.6 KiB
page_title: Advanced contributing page_description: Explains workflows for refactor and design proposals page_keywords: contribute, project, design, refactor, proposal
Advanced contributing
In this section, you learn about the more advanced contributions you can make. They are advanced because they have a more involved workflow or require greater programming experience. Don't be scared off though, if you like to stretch and challenge yourself, this is the place for you.
This section gives generalized instructions for advanced contributions. You'll read about the workflow but there are not specific descriptions of commands. Your goal should be to understand the processes described.
At this point, you should have read and worked through the earlier parts of the project contributor guide. You should also have made at least one project contribution.
Refactor or cleanup proposal
A refactor or cleanup proposal changes Docker's internal structure without altering the external behavior. To make this type of proposal:
-
Fork
docker/docker
. -
Make your changes in a feature branch.
-
Sync and rebase with
master
as you work. -
Run the full test suite.
-
Submit your code through a pull request (PR).
The PR's title should have the format:
Cleanup: short title
If your changes required logic changes, note that in your request.
-
Work through Docker's review process until merge.
Design proposal
A design proposal solves a problem or adds a feature to the Docker software. The process for submitting design proposals requires two pull requests, one for the design and one for the implementation.
The important thing to notice is that both the design pull request and the implementation pull request go through a review. In other words, there is considerable time commitment in a design proposal; so, you might want to pair with someone on design work.
The following provides greater detail on the process:
-
Come up with an idea.
Ideas usually come from limitations users feel working with a product. So, take some time to really use Docker. Try it on different platforms; explore how it works with different web applications. Go to some community events and find out what other users want.
-
Review existing issues and proposals to make sure no other user is proposing a similar idea.
The design proposals are all online in our GitHub pull requests.
-
Talk to the community about your idea.
We have lots of community forums where you can get feedback on your idea. Float your idea in a forum or two to get some commentary going on it.
-
Fork
docker/docker
and clone the repo to your local host. -
Create a new Markdown file in the area you wish to change.
For example, if you want to redesign our daemon create a new file under the
daemon/
folder. -
Name the file descriptively, for example
redesign-daemon-proposal.md
. -
Write a proposal for your change into the file.
This is a Markdown file that describes your idea. Your proposal should include information like:
- Why is this changed needed or what are the use cases?
- What are the requirements this change should meet?
- What are some ways to design/implement this feature?
- Which design/implementation do you think is best and why?
- What are the risks or limitations of your proposal?
This is your chance to convince people your idea is sound.
-
Submit your proposal in a pull request to
docker/docker
.The title should have the format:
Proposal: short title
The body of the pull request should include a brief summary of your change and then say something like "See the file for a complete description".
-
Refine your proposal through review.
The maintainers and the community review your proposal. You'll need to answer questions and sometimes explain or defend your approach. This is chance for everyone to both teach and learn.
-
Pull request accepted.
Your request may also be rejected. Not every idea is a good fit for Docker. Let's assume though your proposal succeeded.
-
Implement your idea.
Implementation uses all the standard practices of any contribution.
- fork
docker/docker
- create a feature branch
- sync frequently back to master
- test as you go and full test before a PR
If you run into issues, the community is there to help.
- fork
-
When you have a complete implementation, submit a pull request back to
docker/docker
. -
Review and iterate on your code.
If you are making a large code change, you can expect greater scrutiny during this phase.
-
Acceptance and merge!
About the Advanced process
Docker is a large project. Our core team gets a great many design proposals. Design proposal discussions can span days, weeks, and longer. The number of comments can reach the 100s. In that situation, following the discussion flow and the decisions reached is crucial.
Making a pull request with a design proposal simplifies this process:
- you can leave comments on specific design proposal line
- replies around line are easy to track
- as a proposal changes and is updated, pages reset as line items resolve
- Github maintains the entire history
While proposals in pull requests do not end up merged into a master repository, they provide a convenient tool for managing the design process.