-
-
Notifications
You must be signed in to change notification settings - Fork 843
Restore 2 weeks inactive label
#8177
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
Restore 2 weeks inactive label
#8177
Conversation
|
Want to review this pull request? Take a look at this documentation for a step by step guide! From your project repository, check out a new branch and test the changes. |
ryanfkeller
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @t-will-gillis,
Your changes to label-directory.json and to HFLA Repo Labels make sense to me for solving the linked issue of restoring the "2 weeks inactive" label back to its previous state! Thanks also for including a screenshot of the label settings. I approve.
I'm new to GH Actions, so could I double check with you my understanding of what all happened to cause this issue and how you went about solving it? If I traced things correctly...
- Someone accidentally deleted the "2 weeks inactive" label on GH, triggering the GH Action to set that label's ID to 999s in
label-directory.jsonto indicate it needs review. - Someone tried to correct the issue by re-adding a new "2 weeks inactive" label on GH, which triggered the GH Action again. However, the JS script it calls is designed to auto-generate the
labelKeyfor new labels based on the description. This meant thatlabel-directory.jsonwound up with a new auto-genned entry "New-2WeeksInactive" that had the ID for the new GH label, while still keeping the original "statusInactive2" with the invalid ID. - You manually updated
label-directory.jsonto give the original "statusInactive2" the new valid ID that links to the new GH label (since the old one is deleted). Then you deleted the extraneous auto-genned entry. - You then updated the label color (and description?) on GH. This triggered a GH Action, but since you didn't change the label name, the triggered script didn't re-modify
label-directory.json.
Is that pretty much right? Thanks!
|
Hey @ryanfkeller thank you for the review and the comments. I will try not to go into too much detail here- feel free to ask for additional information. Also, if you would like to learn more about GHAs or do an issue, there are some you could work on. We created the "Update Label Directory" GHA among other reasons because when someone deletes a label it is also deleted from all issues with that label.
There is an issue #7769 to add to the "Update Label Directory" workflow so that when there is an accidental deletion like this, we are notified and given a list of which issues were un-labeled during the deletion. Only certain labels are regenerated, and the rest we need to add back manually. Thanks again and hope this helps! |
|
Availability: after 1pm (pacific) |
hoanhua14
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @t-will-gillis ,
Thank you for the clarification in the comment, it helps me understand HfLA's workflow better. I've approved the PR, thank you!
kdaca19xx
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Things Done Well
- The pull request done with the correct branch.
- There's a linked issue. I understood it way better after reading ryanfkeller's post and t-will-gillis' response to them, so a huge thanks to the two of you.
- The changes in the Files Changed tab look good.
Suggestions
n/a
Fixes #8176
What changes did you make?
2 weeks inactivelabel and replaced the labelId from the deleted label.Why did you make the changes (we will use this info to test)?
2 weeks inactiveCodeQL Alerts
After the PR has been submitted and the resulting GitHub actions/checks have been completed, developers should check the PR for CodeQL alert annotations.
Check the PR's comments. If present on your PR, the CodeQL alert looks similar as shown
Please let us know that you have checked for CodeQL alerts. Please do not dismiss alerts.
Instructions for resolving CodeQL alerts
If CodeQL alert/annotations appear, refer to How to Resolve CodeQL alerts.
In general, CodeQL alerts should be resolved prior to PR reviews and merging
Screenshots
Restored label settings