-
Notifications
You must be signed in to change notification settings - Fork 217
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
Refresh GitHub issue and PR templates #2808
Conversation
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.
Hi @rvl, this seems like a great improvement!
I've added a few comments and suggestions.
description: | | ||
Operating system on which you are running | ||
`cardano-wallet`. Please include the version. For Linux, | ||
please specify your distribution. |
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.
Perhaps uname -a
would provide helpful info here, for those using Linux. On my system, it prints:
Linux neon 4.15.0-153-generic #160-Ubuntu SMP Thu Jul 29 06:54:29 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
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.
Yeah kernel version is not highly relevant to us, but it can give hints about the exact distro version, and the arch of course.
- Own Docker image (please specify) | ||
- Nix | ||
- via Daedalus installer | ||
- Other |
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.
👍🏻
about: | | ||
Users: Please feel welcome to ask for help from the IOHK | ||
Technical Support team. They are knowledgeable about commonly | ||
experienced issues and workarounds. |
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 is great. 👍🏻
GitHub now supports full YAML for issue templates, so hopefully this will help us get better quality submissions.
3db5761
to
52a3d69
Compare
8d9d4c9
to
cfcb41e
Compare
Thanks for the review comments. I have applied them and tested the templates on a forked repo. |
GitHub now supports full YAML for issue templates, so hopefully this will help us get better quality submissions.
Issue Number
None