Skip to content

Commit 5966d1a

Browse files
authored
Merge pull request #5 from aspiers/two-links
add a couple more links to README
2 parents 2585a50 + 8b93a98 commit 5966d1a

File tree

1 file changed

+11
-6
lines changed

1 file changed

+11
-6
lines changed

README.rst

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,17 @@ want. And for as long as AI is an unsolved problem, it is very
6666
unlikely that it will ever develop totally reliable behaviour.
6767
So does that mean ``git-explode`` is useless? Absolutely not!
6868

69-
Firstly, when `best practices for commit hygiene
70-
<https://wiki.openstack.org/wiki/GitCommitMessages>`_ are adhered to,
71-
changes which are strongly logically related should be within the same
72-
commit anyway. So in the example above, a change to a function and
73-
corresponding changes to the tests and/or documentation for that
74-
function should all be within a single commit.
69+
Firstly, when `best practices
70+
<https://crealytics.com/blog/5-reasons-keeping-git-commits-small/>`_
71+
for `commit structuring
72+
<https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes>`_
73+
are adhered to, changes which are strongly logically related should be
74+
placed within the same commit anyway. So in the example above, a
75+
change to a function and corresponding changes to the tests and/or
76+
documentation for that function should all be within a single commit.
77+
(Although this is not the only valid approach; for a more advanced
78+
meta-history grouping mechanism, see `git-dendrify
79+
<https://github.com/bennorth/git-dendrify>`_.)
7580

7681
Secondly, whilst textual independence does not imply logical
7782
independence, the converse is much more commonly true: logical

0 commit comments

Comments
 (0)