Skip to content

Commit 0de04b3

Browse files
committed
CONTRIBUTING: Suggest contributors to split their commits
Signed-off-by: Povilas Kanapickas <povilas@radix.lt>
1 parent d864e19 commit 0de04b3

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

CONTRIBUTING.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,16 @@ Note: Some steps are OPTIONAL but all are RECOMMENDED.
7070
7171
Fixes https://github.com/containers/podman-compose/issues/516
7272
```
73+
- If your commit requires a refactoring, first do the refactoring and
74+
commit it separately before starting feature work. This makes the
75+
pull request easier to review. Additionally, pull request will be
76+
less risky, because if it breaks something, it's way easier to
77+
isolate the offending code, understand what's broken and fix it.
78+
Due to the latter reason it's best to commit in as many independent
79+
commits as reasonable.
80+
81+
This will result in pull requests being merged much faster.
82+
7383
9. Open a pull request to `containers/podman-compose` and wait for a maintainer to review your work.
7484

7585
## Adding new commands

0 commit comments

Comments
 (0)