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

Ensure stale github actions workflow only runs for particular issue labels #44910

Merged
merged 1 commit into from
Oct 13, 2022

Conversation

talldan
Copy link
Contributor

@talldan talldan commented Oct 13, 2022

What?

A bug was introduced in #44754, whereby stale workflows that are supposed to run for issues with particular labels were run on all issues.

This PR remedies the problem.

I guess we'll have to find a way to go back through all the issues that 'Stale' was accidentally applied to and remove the labels. Maybe a script that removes every stale label added in the last 24 hours?

How?

Ensures the only-labels property is set.

Looks like some automatic formatting was also applied to the yml file 🤔

Testing Instructions

There is no way to test it, we'll just have to wait and see 😎

@talldan talldan added the [Type] Project Management Meta-issues related to project management of Gutenberg label Oct 13, 2022
@talldan talldan self-assigned this Oct 13, 2022
stale-issue-message: ${{ matrix.message }}
days-before-stale: ${{ matrix.days-before-stale }}
days-before-close: ${{ matrix.days-before-close }}
only-labels: ${{ matrix.only-labels }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

From what I understand, an unset value is ok - https://github.com/actions/stale#only-labels:

If unset (or an empty string), this option will not alter the stale workflow.

Copy link
Contributor

@andrewserong andrewserong left a comment

Choose a reason for hiding this comment

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

Looks worth trying to me! From a quick 👀 at #44754, it seems that matrix.only-labels should contain the desired value, and only-labels is the right place for it to be assigned to.

Copy link
Member

@ramonjd ramonjd left a comment

Choose a reason for hiding this comment

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

Looks legit according to the docs. Worth a shot. 😄

@talldan talldan merged commit 36d59fc into trunk Oct 13, 2022
@talldan talldan deleted the fix/stale-issue-workflows branch October 13, 2022 04:13
@github-actions github-actions bot added this to the Gutenberg 14.4 milestone Oct 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Project Management Meta-issues related to project management of Gutenberg
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants