Skip to content

Commit

Permalink
Merge bitcoin#8915: Add copyright/patent issues to possible NACK reasons
Browse files Browse the repository at this point in the history
36f60a5 Add copyright/patent issues to possible NACK reasons (Peter Todd)
  • Loading branch information
MarcoFalke authored and codablock committed Jan 12, 2018
1 parent dcc63e6 commit 552fafd
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,9 @@ language is used within pull-request comments:

- ACK means "I have tested the code and I agree it should be merged";
- NACK means "I disagree this should be merged", and must be accompanied by
sound technical justification. NACKs without accompanying reasoning may be disregarded;
sound technical justification (or in certain cases of copyright/patent/licensing
issues, legal justification). NACKs without accompanying reasoning may be
disregarded;
- utACK means "I have not tested the code, but I have reviewed it and it looks
OK, I agree it can be merged";
- Concept ACK means "I agree in the general principle of this pull request";
Expand Down

0 comments on commit 552fafd

Please sign in to comment.