Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add multiple issue templates #22215

Closed
wants to merge 4 commits into from
Closed
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions .github/ISSUE_TEMPLATE/---bug-report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
---
name: "\U0001F41B Bug report"
about: Create a report to help us improve

---

<!--
Thank you for reporting a possible bug in Node.js.

Please fill in as much of the template below as you're able.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: able. -> able to. + same throughout

Copy link
Member

@Trott Trott Aug 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: able. -> able to. + same throughout

I'm mildly opposed to adding to.

If ending with able seems awkward or unclear, maybe as much as you're able -> as much as you can?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 on as much as you can. Otherwise I would also stumble upon the sentence.


Version: output of `node -v`
Platform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)
Subsystem: if known, please specify the affected core module name

If possible, please provide code that demonstrates the problem, keeping it as
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: provide code -> provide the code

Copy link
Member

@Trott Trott Aug 10, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm mildly opposed to this nit too. Just provide code.

simple and free of external dependencies as you are able.
-->

* **Version**:
* **Platform**:
* **Subsystem**:

<!-- Please provide more details below this comment. -->
20 changes: 20 additions & 0 deletions .github/ISSUE_TEMPLATE/---feature-request.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
---
name: "\U0001F680 Feature request"
about: Suggest an idea for this project

---

<!--
Thank you for suggesting an idea to make Node.js better.

Please fill in as much of the template below as you're able.
-->

**Is your feature request related to a problem? Please describe.**
Please describe the problem you are trying to solve.

**Describe the solution you'd like**
Please describe the desired behavior.

**Describe alternatives you've considered**
Please describe alternative solutions or features you have considered.
8 changes: 8 additions & 0 deletions .github/ISSUE_TEMPLATE/---found-a-problem-with-nodejs-org-.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
---
name: "\U0001F310 Found a problem with nodejs.org?"
about: Please file an issue in the Node.js website repo.

---

If you have a question, suggestion or issue regarding our website,
please post it in https://github.com/nodejs/nodejs.org!
8 changes: 8 additions & 0 deletions .github/ISSUE_TEMPLATE/---need-help-with-node-js-.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
---
name: "⁉️ Need help with Node.js?"
about: Please file an issue in our help repo.

---

If you have a question about Node.js that is not a bug report or feature
request, please post it in https://github.com/nodejs/help!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe also add in this and the node.js org one that this will be closed if opened?