-
Notifications
You must be signed in to change notification settings - Fork 36
Update issue template #137
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
Update issue template #137
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #137 +/- ##
==========================================
+ Coverage 75.08% 79.41% +4.32%
==========================================
Files 17 17
Lines 602 612 +10
Branches 59 59
==========================================
+ Hits 452 486 +34
+ Misses 150 126 -24 |
@anutosh491 @JohanMabille @alexander-penev can one you please review this PR and the other one linked? |
89c3b8d
to
b5df3bf
Compare
@JohanMabille can you take another look at this PR? I fixed all the issues you raised except switching to Github labels. With regards to the Github labels issue I plan to add a workflow which automatically tags each new issue a 'Needs Review' label (needs creating if you could do this for me since I don't have the permissions to do this). It will then be upto the person who reviews the issue to add the correct labels based on the description of the issue. |
Can you try again to create a label? I added you to the repo. |
Thank you I am now able to add a label :) |
@JohanMabille I have now addressed all your issues with this PR and its ready for another review. |
Can I suggest to use the label "Needs triage" instead of "Needs review" for new issues? It is more widespread in OSS repos (at least on GH). otherwise, LGTM. |
@JohanMabille done. |
Thanks! |
This PR updates the issue template for the repo to complement the PR which adds a pull request template here #118