Skip to content

Conversation

@wileyj
Copy link
Contributor

@wileyj wileyj commented Dec 1, 2025

long running nit i had for the workflow cleanup job - it would delete release workflows and i think at a minimum those are worth keeping or at least keeping for much longer since stacks-core is released infrequently.

this change addresses that nit so it will not delete ci.yml workflows for a release.

after i tested this change though, i noticed another field in the actions json that i think might make the jq easier to read:
"event": "workflow_dispatch",

so the filter i currently have of:

| select((.head_branch | test("release/")) and (.name | test("release/") | not) or (.head_branch | test("release/") | not) )

may be more easily written like:

select((.head_branch | test("release/") | not) and ( .event != "workflow_dispatch" ))

untested, but i think it should work. alternatively, the more verbose select stmt proposed in this PR also would work.

@wileyj wileyj marked this pull request as ready for review December 1, 2025 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant