diff options
author | Rich Trott <rtrott@gmail.com> | 2016-05-30 08:32:12 +0300 |
---|---|---|
committer | Rod Vagg <rod@vagg.org> | 2016-06-02 15:42:05 +0300 |
commit | e93198e86df1cd39754759173f9d275f2c5d18dd (patch) | |
tree | 14c15915b1f9d5e061cbce432036100be0d7f7b1 | |
parent | 8b720c4582e7c709325592f62dd93e2807ab0d95 (diff) |
doc: edit pull request template
Move checklist instructions closer to the checklist. Trim unnecessary
words.
PR-URL: https://github.com/nodejs/node/pull/7058
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
-rw-r--r-- | .github/PULL_REQUEST_TEMPLATE.md | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index cce9351dc1e..bc5ba6fafcc 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,9 +1,7 @@ <!-- -Thank you for your pull request. Please review below requirements and walk -through the checklist. You can 'tick' a box by using the letter "x": [x]. +Thank you for your pull request. Please review below requirements. -Run the test suite with: `make -j4 test` on UNIX or `vcbuild test nosign` on -Windows. +Run tests with `make -j4 test` on UNIX or `vcbuild test nosign` on Windows. If this aims to fix a regression or you’re adding a feature, make sure you also write a test. If possible, include a benchmark that quantifies your changes. @@ -13,7 +11,7 @@ https://github.com/nodejs/node/blob/master/CONTRIBUTING.md --> ##### Checklist -<!-- remove lines that do not apply to you --> +<!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [ ] tests and code linting passes - [ ] a test and/or benchmark is included |