diff options
author | Rich Trott <rtrott@gmail.com> | 2021-01-02 17:34:14 +0300 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2021-01-04 19:25:21 +0300 |
commit | 551251e0291880058c87c87f48fd3de92d51a940 (patch) | |
tree | 8b496ebcbd8e5ab884e2835d0aae9c1b8b3bd2c2 /.github | |
parent | 8b8620d580314050175983402dfddf2674e8e22a (diff) |
doc: simplify pull request template
Replace checklist with a list. Clarify when things like tests and
benchmarks are needed.
PR-URL: https://github.com/nodejs/node/pull/36739
Reviewed-By: Zeyu Yang <himself65@outlook.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Yash Ladha <yash@yashladha.in>
Diffstat (limited to '.github')
-rw-r--r-- | .github/PULL_REQUEST_TEMPLATE.md | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 66efca5cd00..56632fda4c6 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,21 +1,15 @@ <!-- -Thank you for your pull request. Please provide a description above and review -the requirements below. +Before submitting a pull request, please read +https://github.com/nodejs/node/blob/master/CONTRIBUTING.md. -Bug fixes and new features should include tests and possibly benchmarks. +Commit message formatting guidelines: +https://github.com/nodejs/node/blob/master/doc/guides/contributing/pull-requests.md#commit-message-guidelines -Contributors guide: https://github.com/nodejs/node/blob/master/CONTRIBUTING.md ---> - -##### Checklist -<!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> +For code changes: +1. Include tests for any bug fixes or new features. +2. Update documentation if relevant. +3. Ensure that `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes. -- [ ] `make -j4 test` (UNIX), or `vcbuild test` (Windows) passes -- [ ] tests and/or benchmarks are included -- [ ] documentation is changed or added -- [ ] commit message follows [commit guidelines](https://github.com/nodejs/node/blob/master/doc/guides/contributing/pull-requests.md#commit-message-guidelines) - -<!-- Developer's Certificate of Origin 1.1 By making a contribution to this project, I certify that: |