-
Notifications
You must be signed in to change notification settings - Fork 4
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
Created issue templates #259
Conversation
Created issue templates in an attempt to standardize/add convention to issues. Hopefully will make them more organizable and readable!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the issue template idea is really good, should encourage folks to add more details. I left a few comments, the steps to reproduce comment is a light suggestion so feel free to ignore it, but the version info might be a bit more helpful. Thanks for putting this together!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, thanks for doing this @emprzy ! Just suggested some step to help the user.
As a general point, a problem is that issues are underused vers slack messages (direct, fast) so we should be careful not to increase the friction opening one. A bad issue is already better than a good slack DM that only two person can read.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great. I added some to the feature request. Also agree with @jcblemai about adding commands to make it easier.
Adding backticks around CL commands
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, can't wait until these are setup.
Implementing Tim's suggestions
Implementing Tim's suggestions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great. this will be very helpful
Note: This PR got re-opened because it was requested that I add mandatory labels for certain categories (e.g., |
Created issue templates in an attempt to standardize/add convention to issues. Hopefully will make them more organizable and readable! To address issue #258