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 "needs triage" label to issue templates #8643

Merged

Conversation

triallax
Copy link
Contributor

What is it?

  • Meta improvement to the project (dev facing)

Description of the changes in your PR

Add a "needs triage" label to the issue templates.

This label would make it easier for issue triagers to know what they haven't triaged yet.

Due diligence

This label would make it easier for issue triagers to know what they
haven't triaged yet.
@triallax triallax force-pushed the add-needs-triage-label-to-issue-templates branch from e09a9fa to 298f660 Compare July 17, 2022 18:00
@triallax triallax changed the title Adds "needs triage" label to issue templates Add "needs triage" label to issue templates Jul 17, 2022
@TobiGr
Copy link
Contributor

TobiGr commented Jul 18, 2022

What about a bot / workflow which automatically adds the labels when someone comments on the issue?
I'd suggest to also remove the label automatically when a team member adds a comment.

@AudricV AudricV added the meta Related to the project but not strictly to code label Jul 18, 2022
@triallax
Copy link
Contributor Author

triallax commented Jul 18, 2022

@TobiGr "needs triage" in this context just means "a team member needs to add labels to this issue and perhaps leave a comment." Therefore, I think such an action wouldn't make much sense.

I'd suggest to also remove the label automatically when a team member adds a comment.

@opusforlife2 did suggest that to me, but I believe that it's not worth it when the team member can just remove the label manually. It's also important to note that in most cases, the team member would be adding labels to the issue anyway, so they only need a little more effort to remove the "needs triage" label.

@opusforlife2
Copy link
Collaborator

@mhmdanas What is this waiting for?

@triallax
Copy link
Contributor Author

Perhaps for @TobiGr's approval.

@triallax triallax merged commit 81c4b82 into TeamNewPipe:dev Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta Related to the project but not strictly to code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants