-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 templates #16213
Add issue templates #16213
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.
Hmm, not sure. I'd rather just go back to delete the issue template again or we dedicate time to design issue templates for ruff:
- new rule
- bug (should mention the rule, if applicable)
but that may require some more thought and I'm not sure it's worth it.
I reported the issue on their feedback discussion https://github.com/orgs/community/discussions/148713#discussioncomment-12225604 |
I'm not sure I agree it's really an either or here? Adding the initial templates is still a step forward. Adding a "new rule" template is another step forward. |
I agree with Micha that this might require some more thought but I'd be ok with including "Bug report", "Feature request", and a "Blank issue". For Ruff, there are two places where users can ask questions - discussions or issue tracker. We haven't provided any strong recommendation on where to ask questions and we leave it to the user wherever they feel comfortable. For this reason, I'm not sure if we should include a "Question" template. It might be useful to have a single place where we collect all these questions. I very much prefer discussions for user queries because of separate threads. So, we could possibly replace the "Question" template with redirecting the users to the discussion or include the following two links in the "Question" template:
|
7b7f32e
to
76f699c
Compare
Hmm. I'm sort of leaning towards not having to think about this right now ;) I'm not sure if I agree with promoting discussions for questions because questions sometimes turn out to be bugs that then need to be transferred. I also don't track discussions as closely as I do issues. |
Github's now aware of the bug https://github.com/orgs/community/discussions/148713#discussioncomment-12226081 |
I think we should just bias towards action here because this is very low-cost to iterate on and the current UX is clearly worse. |
I don't think transferring should be much of an issue as GitHub provides a one-click solution to this. My main motivation for keeping questions in a single place is for visibility and provide an easy way to search through existing resolved queries. This is not to say to transfer all existing issues but just to divert users to use a single place to store all queries. |
I'm going to hand this over to @dhruvmanila — I don't think I can advocate for any particular solution over here. |
* Use "Rule request" instead of "Feature request" * Remove "Example" input in the (now changed) feature request template * Remove "Python version" input in the bug report template * Ask users to search through existing issues and possibly provide a playground link
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'm still undecided on this. I think what we have now works well and I see little reason for changing it. But I also don't want to stay in the way but:
- Let's add a Blank issue entry. There are many more reasons other than questions, bug reports, and new rules (e.g. formatter style, rule changes, rule removals, etc)
- I'd prefer to keep adding labels manually. There are often bug reports that are none. I also use the labels to know if someone from the team looked at the issue. We'd lose that bit of information by auto labelling issues.
Are you suggesting an explicit entry because it is automatically created (refer to the screenshot in the PR description)?
That seems reasonable, I'll remove it. A potential solution would be to add a "needs-triage" label explicitly but we could leave that for future as we decided (in the past) to not move forward with it. This does mean that the main benefit of using issue templates would be to provide suggestions based on what the user is intending to open an issue about. For example, if it's a bug report, we're asking them to check existing issues and optionally include the playground link for better reproduction. |
Preview link to try out: https://github.com/dhruvmanila/ruff-issue-templates/issues |
Sorry, I didn't realize it was there. There are so many options!
Yeah, that's fair. At least if we use the same fields. We could consider explicitly asking for the rule code for bug reports. |
Yeah, we could add multiple fields based on what's the most useful information that's required but I think it's best to iterate on it in follow-ups. |
* main: (38 commits) [red-knot] Use arena-allocated association lists for narrowing constraints (#16306) [red-knot] Rewrite `Type::try_iterate()` to improve type inference and diagnostic messages (#16321) Add issue templates (#16213) Normalize inconsistent markdown headings in docstrings (#16364) [red-knot] Better diagnostics for method calls (#16362) [red-knot] Add argfile and windows glob path support (#16353) [red-knot] Handle pipe-errors gracefully (#16354) Rename `venv-path` to `python` (#16347) [red-knot] Fixup some formatting in `infer.rs` (#16348) [red-knot] Restrict visibility of more things in `class.rs` (#16346) [red-knot] Add diagnostic for class-object access to pure instance variables (#16036) Add `per-file-target-version` option (#16257) [PLW1507] Mark fix unsafe (#16343) [red-knot] Add a test to ensure that `KnownClass::try_from_file_and_name()` is kept up to date (#16326) Extract class and instance types (#16337) Re-order changelog entries for 0.9.7 (#16344) [red-knot] Add support for `@classmethod`s (#16305) Update Salsa (#16338) Update Salsa part 1 (#16340) Upgrade Rust toolchain to 1.85.0 (#16339) ...
Follows #15651
Preview: https://github.com/dhruvmanila/ruff-issue-templates/issues
GitHub made the interface for single-template repositories worse. While they might fix it, it encouragement to just do this work. They still haven't fixed the teeny tiny emojis which makes me think this won't be fixed quickly.
Before:
After: