[tor-commits] [sbws/master] Add section git workflow, simplify commits

juga at torproject.org juga at torproject.org
Tue Sep 25 09:48:10 UTC 2018


commit 57df2a0ab4e8f704c893ba5a79dfcd5611eb5345
Author: juga0 <juga at riseup.net>
Date:   Fri Sep 14 20:07:01 2018 +0000

    Add section git workflow, simplify commits
    
    the reference to how to write commit messages should be enough.
---
 CONTRIBUTING.rst | 45 +++++----------------------------------------
 1 file changed, 5 insertions(+), 40 deletions(-)

diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst
index b61e220..586bc13 100644
--- a/CONTRIBUTING.rst
+++ b/CONTRIBUTING.rst
@@ -101,15 +101,13 @@ Timestamps must be in UTC. It is prefered to use ``datetime`` objects or
 Unix timestamps. Timestamps read by the user should be always formatted in
 `ISO 8601 <https://en.wikipedia.org/wiki/ISO_8601>`_
 
-Git and GitHub Guidelines
-=========================
+Git workflow
+------------
 
-**Write good commit messages** that at least follow the spirit of
-:ref:`this example <commit-msg>`.
+Commits
+~~~~~~~~
 
-Strive to **write many small commits** each containing an atomic change instead
-of one large mega-commit. This not only makes code review easier, but it also
-makes commits that show up in ``git blame`` 10 years from now make more sense.
+Commit messages should follow the `Tim Pope`_ recommendations.
 
 **Prefer a rebase workflow instead of merge**. Incorporating PRs should be done
 with fast-forward merge, if easily possible. The larger the topic branch, the
@@ -125,39 +123,6 @@ may get overwritten (for example: "transient\_" prefix). Instead of overwriting
 a branch, add a version suffix (for example: "_02").
 
 
-.. _commit-msg:
-
-Example commit message
-======================
-
-With thanks to `Tim Pope`_:
-
-
-::
-
-    Capitalized, short (50 chars or less) summary
-
-    More detailed explanatory text, if necessary.  Wrap it to about 72
-    characters or so.  In some contexts, the first line is treated as the
-    subject of an email and the rest of the text as the body.  The blank
-    line separating the summary from the body is critical (unless you omit
-    the body entirely); tools like rebase can get confused if you run the
-    two together.
-
-    Write your commit message in the imperative: "Fix bug" and not "Fixed bug"
-    or "Fixes bug."  This convention matches up with commit messages generated
-    by commands like git merge and git revert.
-
-    Further paragraphs come after blank lines.
-
-    - Bullet points are okay, too
-
-    - Typically a hyphen or asterisk is used for the bullet, followed by a
-      single space, with blank lines in between, but conventions vary here
-
-    - Use a hanging indent
-
-
 
 .. _pull request: https://github.com/pastly/simple-bw-scanner/compare
 





More information about the tor-commits mailing list