mirror of
https://github.com/monero-project/monero.git
synced 2024-12-27 04:49:25 -05:00
CONTRIBUTING.md: mention git add -p as a way to avoid spurious changes
This commit is contained in:
parent
49ce59462a
commit
631b35e7db
@ -25,6 +25,17 @@ modifying is encourgaged. Proper squashing should be done (eg, if
|
|||||||
you're making a buggy patch, then a later patch to fix the bug,
|
you're making a buggy patch, then a later patch to fix the bug,
|
||||||
both patches should be merged).
|
both patches should be merged).
|
||||||
|
|
||||||
|
If you've made random unrelated changes (either because your editor
|
||||||
|
is annoying or you made them for other reasons), you can select
|
||||||
|
what changes go into the coming commit using git add -p, which
|
||||||
|
walks you through all the changes and asks whether or not to
|
||||||
|
include this particular change. This helps create clean patches
|
||||||
|
without any irrelevant changes. git diff will show you the changes
|
||||||
|
in your tree. git diff --cached will show what is currently staged
|
||||||
|
for commit. As you add hunks with git add -p, those hunks will
|
||||||
|
"move" from the git diff output to the git diff --cached output,
|
||||||
|
so you can see clearly what your commit is going to look like.
|
||||||
|
|
||||||
## Commits and Pull Requests
|
## Commits and Pull Requests
|
||||||
|
|
||||||
Commit messages should be sensible. That means a subject line that
|
Commit messages should be sensible. That means a subject line that
|
||||||
|
Loading…
Reference in New Issue
Block a user