doc: Put PR template in comments
This prevents the common annoyance of the text being included into PRs accidentally.
This commit is contained in:
parent
36604b4ef5
commit
203a67d21f
1 changed files with 9 additions and 1 deletions
10
.github/PULL_REQUEST_TEMPLATE.md
vendored
10
.github/PULL_REQUEST_TEMPLATE.md
vendored
|
@ -1,9 +1,14 @@
|
||||||
|
<!--
|
||||||
|
*** Please remove the following help text before submitting: ***
|
||||||
|
|
||||||
Pull requests without a rationale and clear improvement may be closed
|
Pull requests without a rationale and clear improvement may be closed
|
||||||
immediately.
|
immediately.
|
||||||
|
-->
|
||||||
|
|
||||||
|
<!--
|
||||||
Please provide clear motivation for your patch and explain how it improves
|
Please provide clear motivation for your patch and explain how it improves
|
||||||
Bitcoin Core user experience or Bitcoin Core developer experience
|
Bitcoin Core user experience or Bitcoin Core developer experience
|
||||||
significantly.
|
significantly:
|
||||||
|
|
||||||
* Any test improvements or new tests that improve coverage are always welcome.
|
* Any test improvements or new tests that improve coverage are always welcome.
|
||||||
* All other changes should have accompanying unit tests (see `src/test/`) or
|
* All other changes should have accompanying unit tests (see `src/test/`) or
|
||||||
|
@ -24,8 +29,11 @@ significantly.
|
||||||
is often a subjective matter. Unless they are explicitly mentioned to be
|
is often a subjective matter. Unless they are explicitly mentioned to be
|
||||||
preferred in the [developer notes](/doc/developer-notes.md), stylistic code
|
preferred in the [developer notes](/doc/developer-notes.md), stylistic code
|
||||||
changes are usually rejected.
|
changes are usually rejected.
|
||||||
|
-->
|
||||||
|
|
||||||
|
<!--
|
||||||
Bitcoin Core has a thorough review process and even the most trivial change
|
Bitcoin Core has a thorough review process and even the most trivial change
|
||||||
needs to pass a lot of eyes and requires non-zero or even substantial time
|
needs to pass a lot of eyes and requires non-zero or even substantial time
|
||||||
effort to review. There is a huge lack of active reviewers on the project, so
|
effort to review. There is a huge lack of active reviewers on the project, so
|
||||||
patches often sit for a long time.
|
patches often sit for a long time.
|
||||||
|
-->
|
||||||
|
|
Loading…
Reference in a new issue