diff options
| author | MarcoFalke <[email protected]> | 2016-10-14 09:52:29 +0200 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2016-10-14 09:53:22 +0200 |
| commit | 03dd707dc027fbf6f24120213f8eb66571600374 (patch) | |
| tree | 1cdb1262d476d1bf3984191b0324091914342302 | |
| parent | Merge #8817: update bitcoin-tx to output witness data (diff) | |
| parent | Add copyright/patent issues to possible NACK reasons (diff) | |
| download | discoin-03dd707dc027fbf6f24120213f8eb66571600374.tar.xz discoin-03dd707dc027fbf6f24120213f8eb66571600374.zip | |
Merge #8915: Add copyright/patent issues to possible NACK reasons
36f60a5 Add copyright/patent issues to possible NACK reasons (Peter Todd)
| -rw-r--r-- | CONTRIBUTING.md | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index fd1a912c4..06fcd8dd8 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -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"; |