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

Readme: Add ISSUE_TEMPLATE with issue guidelines to repository #1104

Merged
merged 2 commits into from Feb 27, 2017

Conversation

Copy link
Contributor

@platinumazure platinumazure commented Feb 20, 2017

I thought it might be a good idea to take advantage of GitHub's support for issue templates to try to ensure we get useful issue reports.

I wrote this template very quickly and it probably could use serious copy editing. Please feel free to either make suggestions here or just push directly to my branch to help improve this. Or, if we don't think we'll get value out of this, feel free to close this PR.

Copy link
Member

@trentmwillis trentmwillis left a comment

Overall I think this is a great idea, some suggestions for tweaks though


## Tell us about your runtime:

* **OS**:
Copy link
Member

@trentmwillis trentmwillis Feb 20, 2017

Choose a reason for hiding this comment

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

I think we should leave this out to minimize friction. We rarely ever get cases where the OS influences behavior.


* **OS**:
* **QUnit version**:
* **How are you running QUnit? (e.g., browser, Grunt)**:
Copy link
Member

@trentmwillis trentmwillis Feb 20, 2017

Choose a reason for hiding this comment

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

I think this may potentially conflate two separate questions/pieces of information:

  1. What environment are you running QUnit in? (e.g., browser, node)
  2. How are you running QUnit? (e.g., testem, grunt, script, etc.)


## What are you trying to do?

Code that reproduces the problem:
Copy link
Member

@trentmwillis trentmwillis Feb 20, 2017

Choose a reason for hiding this comment

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

We should likely add a comment to the effect of "Please provide an exact reproduction of your issue if possible. If not, then please provide detailed information about your setup and what you are trying to do."

This is because sometimes developers can't share the actual code (e.g., when working on private projects) and may not have time to get a minimal reproduction setup before filing the issue.

@platinumazure
Copy link
Author

@platinumazure platinumazure commented Feb 20, 2017

@trentmwillis Thanks, I've updated per your suggestions. Let me know if other improvements are needed.

@platinumazure
Copy link
Author

@platinumazure platinumazure commented Feb 22, 2017

Ping @qunitjs/qunit-team, how does this look now? Anything else I can do to improve this?

@platinumazure
Copy link
Author

@platinumazure platinumazure commented Feb 27, 2017

One more friendly ping to @qunitjs/qunit-team, anything else I need to change or can this be merged?

@leobalter
Copy link

@leobalter leobalter commented Feb 27, 2017

I seems fine to me. I'm sorry for the brevity in my answer. I really like to have these templates in here.

@platinumazure
Copy link
Author

@platinumazure platinumazure commented Feb 27, 2017

@leobalter No problem, thanks for commenting.

Copy link
Member

@trentmwillis trentmwillis left a comment

Sorry for the delay, this seems good to me now. Thanks!

@leobalter leobalter merged commit 23012af into qunitjs:master Feb 27, 2017
2 checks passed
@platinumazure
Copy link
Author

@platinumazure platinumazure commented Feb 27, 2017

@platinumazure platinumazure deleted the issue-template branch Feb 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants