summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--SUBMITTING_PATCHES.md14
1 files changed, 9 insertions, 5 deletions
diff --git a/SUBMITTING_PATCHES.md b/SUBMITTING_PATCHES.md
index 0c189247..76c167c3 100644
--- a/SUBMITTING_PATCHES.md
+++ b/SUBMITTING_PATCHES.md
@@ -3,7 +3,7 @@
3# Short Version 3# Short Version
4 4
5 - Make small logical changes. 5 - Make small logical changes.
6 - Provide a meaningful commit message. 6 - [Provide a meaningful commit message][commit-message-style].
7 - Check for coding errors and style nits with flake8. 7 - Check for coding errors and style nits with flake8.
8 - Make sure all code is under the Apache License, 2.0. 8 - Make sure all code is under the Apache License, 2.0.
9 - Publish your changes for review. 9 - Publish your changes for review.
@@ -26,10 +26,11 @@ yourself with the following relevant bits.
26 26
27## Make separate commits for logically separate changes. 27## Make separate commits for logically separate changes.
28 28
29Unless your patch is really trivial, you should not be sending 29Unless your patch is really trivial, you should not be sending out a patch that
30out a patch that was generated between your working tree and your 30was generated between your working tree and your commit head.
31commit head. Instead, always make a commit with complete commit 31Instead, always make a commit with a complete
32message and generate a series of patches from your repository. 32[commit message][commit-message-style] and generate a series of patches from
33your repository.
33It is a good discipline. 34It is a good discipline.
34 35
35Describe the technical detail of the change(s). 36Describe the technical detail of the change(s).
@@ -171,3 +172,6 @@ After you receive a Code-Review+2 from the maintainer, select the Verified
171button on the gerrit page for the change. This verifies that you have tested 172button on the gerrit page for the change. This verifies that you have tested
172your changes and notifies the maintainer that they are ready to be submitted. 173your changes and notifies the maintainer that they are ready to be submitted.
173The maintainer will then submit your changes to the repository. 174The maintainer will then submit your changes to the repository.
175
176
177[commit-message-style]: https://chris.beams.io/posts/git-commit/