-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Bug Triage: add workflow to automatically apply the [Status] Stale
label for issues closed with a keyword.
#80358
Conversation
- new workflow to add stale closure label if warranted on closed issues.
- add control character escaping
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
- name: Set Last Comment | ||
id: get_last_comment | ||
run: | | ||
last_comment=$( echo '${{ steps.get_all_comments.outputs.data }}' | tr -d '[:cntrl:]' | jq -r '.[-1].body' ) |
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.
(I'm not too familiar with the workflow.)
steps.get_all_comments.outputs.data
question: The ID used in the previous step was get_comments
. But, get_all_comments
is being referenced here. Shouldn't the reference be ${{ steps.get_comments.outputs.data }}
?
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.
Good catch, thank you for your eagle-eyed observation!
- fix mismatch between step id
Context: p1691054833018099-slack-CQD1HH4MA
Proposed Changes
This PR adds a GitHub workflow to automatically add the label (shown above) to issues that KitKat closes with the agreed-upon predef.
The aim for this workflow is to make sure we appropriately label issues closed under this Stale issue effort for future reporting purposes.
Testing Instructions
Cannot be tested until merged and running.
That said, I have tested a near-identical copy of the workflow in another repository to make sure it works.
Example issue: Automattic/wp-calypso-test-results#23
Workflow: https://github.com/Automattic/wp-calypso-test-results/blob/trunk/.github/workflows/label.yml
Pre-merge Checklist