Skip to content

Commit aebe990

Browse files
committed
Merge #15794: docs: Clarify PR guidelines w/re documentation
f4a230b docs: Clarify PR guidelines w/re documentation (Carl Dong) Pull request description: PRs should change documentation accordingly when the behaviour of code changes. ACKs for commit f4a230: practicalswift: ACK f4a230b fanquake: utACK f4a230b Tree-SHA512: 6d9d65d7f0f9bc8f324ee16f03169df28fb512c58bb71093128cf16797b25533cdc992bc8757034a99d6fa6423a3129ca7cf2ab2da857535f409a683486fd4ab
2 parents a65fd92 + f4a230b commit aebe990

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

CONTRIBUTING.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -220,6 +220,7 @@ In general, all pull requests must:
220220
- Not break the existing test suite;
221221
- Where bugs are fixed, where possible, there should be unit tests
222222
demonstrating the bug and also proving the fix. This helps prevent regression.
223+
- Change relevant comments and documentation when behaviour of code changes.
223224

224225
Patches that change Bitcoin consensus rules are considerably more involved than
225226
normal because they affect the entire ecosystem and so must be preceded by

0 commit comments

Comments
 (0)