chore: Add action to auto update labels on comments#3194
Merged
michalChrobot merged 21 commits intodevelopfrom Apr 1, 2025
Merged
chore: Add action to auto update labels on comments#3194michalChrobot merged 21 commits intodevelopfrom
michalChrobot merged 21 commits intodevelopfrom
Conversation
EmandM
requested review from
NoelStephensUnity,
chrispope,
fluong6 and
miniwolf
as code owners
michalChrobot
added
stat:awaiting-triage
…nity.netcode.gameobjects into chore/add_auto_labeler
michalChrobot
requested changes
Mar 10, 2025
Member
michalChrobot
left a comment
michalChrobot
left a comment
There was a problem hiding this comment.
I think that it's super nice if we can have this automation (in contrast to running tool manually) but I wonder about few things
- I would argue that after closing the issue some labels (like type: or "stat:imported" or port: or priority: or "regression") can be useful from "history tracking perspective" so I wonder if we could remove all BUT those for example
- With the "autolabel" workflow I believe that at the end we came to the conclusion that "stat:awaiting response" label should be set manually (to avoid adding this label when not really waiting for response but rather commenting on the thread)
What I was thinking in general are the following rules (included in this manual tool):
- Remove "stat:awaiting triage", "stat:awaiting response" and "stat:Investigating" labels from closed issues (this list could be expanded or modified depending on which labels we want to keep/remove)
- When a person is assigned to an issue, the "stat:awaiting triage" label should be removed. IF label “import” or “imported” is not present then label “stat:Investigating” should be added
- When an issue is not assigned to anyone (someone de-assigned) label “stat:awaiting triage” should be added. IF label “import” or “imported” is not present then label “stat:Investigating” should be removed
- When label “import” or “imported” is present then labels “stat:awaiting triage” and “stat:Investigating” should not be present
- When "stat:awaiting response" is present but Issue owner replied to the thread after this label was added, remove it. (This means that we are no longer waiting for the answer)
- When an issue has “stat:awaiting response” tag for two months, add a “stale” tag.
- When the issue has “stale” label for two months (four months in total with no response) report it (somehow) so it may be potentially closed.
Does this make sense? Could something like this be implemented in GitHub workflow?
…hnologies/com.unity.netcode.gameobjects into chore/add_auto_labeler
… from closed issues
michalChrobot
approved these changes
Mar 19, 2025
EmandM
commented
Mar 20, 2025
Member
Author
EmandM
left a comment
EmandM
left a comment
There was a problem hiding this comment.
A couple of notes. Overall this is fantastic!
michalChrobot
added a commit
that referenced
this pull request
Apr 11, 2025
This PR ports all new GitHub Actions that were added to develop branch. This will not have any temporary effect since GitHub Actions utilizes configuration from main branch (currently develop) but this ensures that when we will change main branch to be develop-2.0.0 everything will be working in the same way ## Backport This PR ports changes that were included in #3194 and #3380
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adding Github action that will auto-update labels on Issues.