diff options
| author | Ingela Andin <ingela@erlang.org> | 2020-06-12 14:04:08 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-06-12 14:04:08 +0200 |
| commit | da87db7519e826d64e236fb1f201cf15d63ce07f (patch) | |
| tree | c80d40a2bee6320d508388cfd717875a620caad7 | |
| parent | 07d852f64b1345b239c4ee826dbab8fd1aa59dfa (diff) | |
| parent | 1ae371d56a8c6f25d0bb80cb9b5d54a3866b6712 (diff) | |
| download | erlang-da87db7519e826d64e236fb1f201cf15d63ce07f.tar.gz | |
Merge pull request #2657 from IngelaAndin/ingela/contributions
Clarify how to make contributions
| -rw-r--r-- | CONTRIBUTING.md | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index b754fbd662..068ae0507e 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -109,8 +109,11 @@ Make sure that your branch contains clean commits: * Don't merge `maint` or `master` into your branch. Use `git rebase` if you need to resolve merge conflicts or include the latest changes. -* To make it possible to use the powerful `git bisect` command, make sure that each commit can be -compiled and that it works. +* Each commit should represent a logical change, such as a feature added or bug fixed, and also include relevant changes to documentation and tests. + +* Each commit should compile separately and pass the most relevant test cases. This makes it possible to use the powerful `git bisect` command. + +* Changes to multiple applications should be made in separate commits to facilitate code reviews, unless special circumstances motivates a single commit, such as not breaking the ability to build cleanly. * Check for unnecessary whitespace before committing with `git diff --check`. However, do not fix preexisting whitespace errors in otherwise untouched source lines. |