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 ISSUE_TEMPLATE.md #4474

Merged
merged 1 commit into from
Oct 20, 2016
Merged

Add ISSUE_TEMPLATE.md #4474

merged 1 commit into from
Oct 20, 2016

Conversation

pjcozzi
Copy link
Contributor

@pjcozzi pjcozzi commented Oct 20, 2016

Let's see if this issue template helps. If not, we can change or remove it in a few weeks.

There's lots more it could say, but I want to keep it concise.

Fixes #4443

@hpinkos
Copy link
Contributor

hpinkos commented Oct 20, 2016

👍 thanks @pjcozzi! I think this will make a difference

@hpinkos hpinkos merged commit f9ddb9d into master Oct 20, 2016
@hpinkos hpinkos deleted the template branch October 20, 2016 17:22
@mramato
Copy link
Contributor

mramato commented Oct 20, 2016

@pjcozzi please add a separate pull request that ignores this file from both the release zip and adds it to npmignore. Thanks.

@mramato
Copy link
Contributor

mramato commented Oct 20, 2016

Looks like it is not part of the zip, but it does need to be in npmignore.

@pjcozzi
Copy link
Contributor Author

pjcozzi commented Oct 20, 2016

See #4478 for npmignore

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants