Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit b725437

Browse filesBrowse files
Fishrock123Fishrock123
committed
doc: minor rewording to the GitHub issue/pr templates
- Templates are now a bit more consistent with phrasing. - Simplified a bit. PR-URL: #7403 Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Minwoo Jung <jmwsoft@gmail.com> Reviewed-By: Michaël Zasso <mic.besace@gmail.com>
1 parent 6b1fc63 commit b725437
Copy full SHA for b725437

File tree

Expand file treeCollapse file tree

2 files changed

+10
-11
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

2 files changed

+10
-11
lines changed
Open diff view settings
Collapse file

‎.github/ISSUE_TEMPLATE.md‎

Copy file name to clipboardExpand all lines: .github/ISSUE_TEMPLATE.md
+4-4Lines changed: 4 additions & 4 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
<!--
2-
Thank you for reporting an issue. Please fill in the template below. If unsure
3-
about something, just do as best as you're able.
2+
Thank you for reporting an issue.
3+
Please fill in as much of the template below as you're able.
44
5-
Version: usually output of `node -v`
6-
Platform: either `uname -a` output, or if Windows, version and 32 or 64-bit
5+
Version: output of `node -v`
6+
Platform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)
77
Subsystem: if known, please specify affected core module name
88
99
If possible, please provide code that demonstrates the problem, keeping it as
Collapse file

‎.github/PULL_REQUEST_TEMPLATE.md‎

Copy file name to clipboard
+6-7Lines changed: 6 additions & 7 deletions
  • Display the source diff
  • Display the rich diff
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,22 @@
11
<!--
22
Thank you for your pull request. Please review below requirements.
33
4-
Bug fixes and new features should include tests and possibly a benchmark.
4+
Bug fixes and new features should include tests and possibly benchmarks.
55
66
Contributors guide: https://github.com/nodejs/node/blob/master/CONTRIBUTING.md
77
-->
88

99
##### Checklist
1010
<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->
1111

12-
- [ ] `make -j4 test` (UNIX) or `vcbuild test nosign` (Windows) passes
13-
- [ ] a test and/or benchmark is included
12+
- [ ] `make -j4 test` (UNIX), or `vcbuild test nosign` (Windows) passes
13+
- [ ] tests and/or benchmarks are included
1414
- [ ] documentation is changed or added
15-
- [ ] the commit message follows commit guidelines
16-
15+
- [ ] commit message follows commit guidelines
1716

1817
##### Affected core subsystem(s)
19-
<!-- provide affected core subsystem(s) (like doc, cluster, crypto, etc) -->
18+
<!-- Provide affected core subsystem(s) (like doc, cluster, crypto, etc). -->
2019

2120

2221
##### Description of change
23-
<!-- provide a description of the change below this comment -->
22+
<!-- Provide a description of the change below this comment. -->

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.