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

chore: Add action to auto update labels on comments #3194

Open
wants to merge 18 commits into
base: develop
Choose a base branch
from

Conversation

EmandM
Copy link
Collaborator

@EmandM EmandM commented Jan 8, 2025

Adding Github action that will auto-update labels on Issues.

@michalChrobot michalChrobot added stat:awaiting-triage Status - Awaiting triage from the Netcode team. port:2.x-needed This issue needs to be ported to 2.X branch and removed up-port-for-2.0.0-required stat:awaiting-triage Status - Awaiting triage from the Netcode team. labels Jan 16, 2025
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity left a comment

Choose a reason for hiding this comment

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

👍

@chrispope chrispope removed their request for review February 26, 2025 11:33
@EmandM EmandM requested a review from michalChrobot as a code owner March 8, 2025 00:37
Copy link
Collaborator

@michalChrobot michalChrobot left a comment

Choose a reason for hiding this comment

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

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

  1. 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
  2. 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):

  1. 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)
  2. 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
  3. 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
  4. When label “import” or “imported” is present then labels “stat:awaiting triage” and “stat:Investigating” should not be present
  5. 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)
  6. When an issue has “stat:awaiting response” tag for two months, add a “stale” tag.
  7. 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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
port:2.x-needed This issue needs to be ported to 2.X branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants