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

Switch issue templates to use new-style issue types #13289

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jakelishman
Copy link
Member

Summary

Qiskit is part of the new-style GitHub issues beta1, which gives us access to a special kind of "super" label: the issue "type". We've previously used two special labels ("bug" and "type: feature request") to categorise the resulting issues from the issue template. This commit leans into this new system, setting the issue type, rather than the custom labels.

The new issues also specifies that the templates will be displayed in sorted order by file name. This commit makes that ordering more explicit for us.

Details and comments

If we choose to merge this, I can bulk update all our issues to use the new "types" rather than the magic labels, so we'll have a more consistent experience across our tracker. The new "subissues" system is pushing us towards using these types, so it feels cleaner to centralise.

Footnotes

  1. https://github.blog/changelog/2024-10-01-evolving-github-issues-public-beta/

Qiskit is part of the new-style GitHub issues beta[^1], which gives us
access to a special kind of "super" label: the issue "type".  We've
previously used two special labels ("bug" and "type: feature request")
to categorise the resulting issues from the issue template.  This commit
leans into this new system, setting the issue type, rather than the
custom labels.

The new issues also specifies that the templates will be displayed in
sorted order by file name.  This commit makes that ordering more
explicit for us.

[^1]: https://github.blog/changelog/2024-10-01-evolving-github-issues-public-beta/
@jakelishman jakelishman added type: qa Issues and PRs that relate to testing and code quality Changelog: None Do not include in changelog labels Oct 7, 2024
@jakelishman jakelishman requested a review from a team as a code owner October 7, 2024 14:01
@qiskit-bot
Copy link
Collaborator

One or more of the following people are relevant to this code:

  • @Qiskit/terra-core

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11217189473

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.01%) to 88.883%

Files with Coverage Reduction New Missed Lines %
crates/qasm2/src/lex.rs 3 92.48%
Totals Coverage Status
Change from base Build 11181214888: 0.01%
Covered Lines: 74468
Relevant Lines: 83782

💛 - Coveralls

@eliarbel
Copy link
Contributor

eliarbel commented Oct 7, 2024

I think with the sub-issues feature it also makes sense to add a template and support the Epic type.

@jakelishman
Copy link
Member Author

Eli: is there going to be any advantage to that over just using the "blank issue"? I think of the issue templates as being for external contributors to use slightly more than the core team doing internal tracking, and I can't imagine a situation where an external contributor would be / should be empowered to set us an epic. I'm also not sure there'd be any standard structure to an epic that the form would help with.

@eliarbel
Copy link
Contributor

eliarbel commented Oct 8, 2024

My main motivation is to facilitate organizing tasks in a more hierarchical fashion now that we have the sub-issues feature available. So given a template like the below, I think that having a template Epic issue type in the Create new issue list which explicitly mentions sub-tasking would help that. Of course it's minor and just a convenience tool but still nice to have.

Background
Summary
Subtasks

In any case and regardless, I agree with your statement about epics being more of the core team responsibility to set up.

@jakelishman
Copy link
Member Author

I don't feel strongly, and I feel like we could adjust the documentation of the template to make it clear that an "Epic" template is not intended for general use. If you think that structure's useful, I can write up a template for it (maybe a separate PR, then we can play with it more separately?).

@eliarbel
Copy link
Contributor

eliarbel commented Oct 8, 2024

Sounds good.

@jakelishman
Copy link
Member Author

Eli: do you want to approve / review this PR as-is, then, then I'll get to updating the issue metadata for the repo and make the new template?

@@ -1,6 +1,6 @@
name: 🚀 Feature request
description: Suggest an idea for this project 💡!
labels: ["type: feature request"]
type: Enhancement
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We currently have in the label space type: enhancement and type: feature request, each with different meaning, namely "improve existing functionality" and "suggest or ask new feature", respectively. Assuming we want to have those 2 concepts captured in issue types (I would vote for it) It would be good to preserve this separation by naming the type here with Feature or something similar, but not Enhancement.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have to set the allowed types at the organisation level (if I understand the docs correctly), but that seems easily doable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog: None Do not include in changelog type: qa Issues and PRs that relate to testing and code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants