-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
ADMIN: Create separate issue templates for different usecases #33187
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
Conversation
.github/ISSUE_TEMPLATE/bug_report.md
Outdated
|
||
```python | ||
# Your code here | ||
|
||
``` | ||
|
||
**Note**: Please read [this guide](https://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports) detailing how to provide the necessary information for us to reproduce your bug. |
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.
IMO, this should go before the Code Sample
.github/ISSUE_TEMPLATE/bug_report.md
Outdated
#### Problem description | ||
|
||
[this should explain **why** the current behaviour is a problem and why the expected output is a better solution.] | ||
[this should explain **why** the current behaviour is a problem and why the expected output is a better solution] | ||
|
||
**Note**: We receive a lot of issues on our GitHub tracker, so it is very possible that your issue has been posted before. Please check first before submitting so that we do not have to handle and close duplicates! |
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.
As a suggestion. I think we should change these two **Note**
sections into checkboxes like
- [ ] I have checked that issue has not already been reported
- [ ] I have confirmed this bug exists on the latest version of pandas
- [ ] I have confirmed this bug exists on the master branch of pandas
More succinct than these sections IMO.
Inspired how Numba does their bug reports: numba/numba#4735
|
||
#### Location of the documentation | ||
|
||
[this should provide a description of what documentation you believe needs to be fixed/improved] |
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 we should also encourage a link to the documentation that they want to see improved.
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.
Thanks for reviving this.
|
||
#### Describe the solution you'd like | ||
|
||
[this should provide a description of the feature request, e.g. "`DataFrame.foo` should get a new parameter `bar` that [...]"] |
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've found asking people to "write a docstring" to be helpful in clarifying what people are asking for.
Co-Authored-By: Tom Augspurger <[email protected]>
Co-Authored-By: Joris Van den Bossche <[email protected]>
.github/ISSUE_TEMPLATE/bug_report.md
Outdated
name: Bug Report | ||
about: Create a bug report to help us improve pandas | ||
title: "BUG:" | ||
labels: "" |
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'd say we should use this label:
feature to automatically tag the issue with the appropriate label. For this template Bug
and similarly for the others.
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.
Just making sure before I make the changes, the labels for each file should look like this:
bug_report: Bug
documentation_improvement: Docs
feature_request: Enhancement?
submit_question: "Usage Question"
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 was originally on the fence about automatically adding labels because it could make it harder to distinguish what has been triaged vs. what hasn't, since I currently look for the presence of labels to determine if something has been triaged, and automatically adding labels would make it less clear whether a triage has occurred or not.
A potential workaround here is to also automatically add a new "Needs Triage" label and then remove it as part of the triage process. Or maybe this is only an issue on my part, in which case I can work around it.
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.
feature_request: Enhancement?
We don't have a label for feature requests
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.
We don't have a label for feature requests
We typically use the "Enhancement" label for this, so I think what you've proposed above is good.
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, I've seen a couple variants of this: rapidsai/cudf automatically adds a generic "? - Needs Triage" label, scikit-learn/scikit-learn adds a "Bug: triage" just to bug reports.
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.
Numba has a need triage
label: https://github.com/numba/numba/issues?q=is%3Aissue+is%3Aopen+label%3Aneedtriage
I'd be in favor in autotagging with <Bug/Doc/Enhancement/...> + Need Traige
, and then Need Triage
can be removed once someone has commented (and can be replace with a more specific label like IO CSV
for example)
I've also been making an effort to tag older issue with one of Bug/Doc/Enhancement/... which will be useful in the future for developing metrics about total number of issues per category.
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.
@mroeschke ATM Need Triage
label does not exists.
Should I wait for it to be created? or just edit this pull request to auto-assign the label Need Triage
and the label will be created later.
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 went ahead and just created a Needs Triage
label so it should be available now.
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.
Thank you @mroeschke
Co-Authored-By: Jeremy Schendel <[email protected]>
Co-Authored-By: Jeremy Schendel <[email protected]>
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!
thanks @MomIsBestFriend |
it even pre-sets some labels, nice! |
revival of #31551 originally opened by @jschendel