Skip to content
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

Merged
merged 6 commits into from
Feb 25, 2025
Merged

Add issue templates #16213

merged 6 commits into from
Feb 25, 2025

Conversation

zanieb
Copy link
Member

@zanieb zanieb commented Feb 17, 2025

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:

Screenshot 2025-02-17 at 8 26 08 AM

After:

Screenshot 2025-02-24 at 3 05 35 PM

@zanieb zanieb added the internal An internal refactor or improvement label Feb 17, 2025
Copy link
Member

@MichaReiser MichaReiser left a 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.

@MichaReiser
Copy link
Member

I reported the issue on their feedback discussion https://github.com/orgs/community/discussions/148713#discussioncomment-12225604

@zanieb
Copy link
Member Author

zanieb commented Feb 17, 2025

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.

@dhruvmanila
Copy link
Member

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:

@zanieb
Copy link
Member Author

zanieb commented Feb 17, 2025

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.

This is a great point. We can link to the Discussions page instead.

Screenshot 2025-02-17 at 9 14 53 AM

@MichaReiser
Copy link
Member

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.

@MichaReiser
Copy link
Member

@zanieb
Copy link
Member Author

zanieb commented Feb 17, 2025

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.

@dhruvmanila
Copy link
Member

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.

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.

@zanieb
Copy link
Member Author

zanieb commented Feb 18, 2025

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
@dhruvmanila dhruvmanila changed the title Port uv issue templates to ruff Add issue templates Feb 19, 2025
@dhruvmanila dhruvmanila marked this pull request as ready for review February 24, 2025 08:39
Copy link
Member

@MichaReiser MichaReiser left a 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.

@dhruvmanila
Copy link
Member

  • 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)

Are you suggesting an explicit entry because it is automatically created (refer to the screenshot in the PR description)?

  • 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.

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.

@dhruvmanila
Copy link
Member

Preview link to try out: https://github.com/dhruvmanila/ruff-issue-templates/issues

@MichaReiser
Copy link
Member

Are you suggesting an explicit entry because it is automatically created (refer to the screenshot in the PR description)?

Sorry, I didn't realize it was there. There are so many options!

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.

Yeah, that's fair. At least if we use the same fields. We could consider explicitly asking for the rule code for bug reports.

@dhruvmanila
Copy link
Member

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.

@dhruvmanila dhruvmanila merged commit 1be0dc6 into main Feb 25, 2025
21 checks passed
@dhruvmanila dhruvmanila deleted the zb/ruff-templates branch February 25, 2025 10:59
dcreager added a commit that referenced this pull request Feb 25, 2025
* 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)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal An internal refactor or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants