Skip to content

workflow-runners drops the marola-sea-publish.yml exemption - #172

Merged
brunogbv merged 1 commit into
mainfrom
chore/168-runners-exemption
Oct 17, 2026
Merged

brunogbv merged 1 commit into
mainfrom
chore/168-runners-exemption

Conversation

@brunogbv

Copy link
Copy Markdown
Collaborator

marola-ml removed marola-sea-publish.yml and nothing is self-hosted now, so the exemption
only let a new file of that name reach the runner. No workflow may name self-hosted or an
expression in runs-on/runner; the PR-trigger rule that guarded the exempt file goes with it.
The runners page, the tools table and the README say so.

Closes #168

Tested: workflow_runners.py --self-test red (sea-publish case) then green; devkit-quality.sh rc=0
Co-Authored-By: Claude noreply@anthropic.com

The AC grep for marola-sea-publish still prints two lines, both in the self-test (scripts/workflow_runners.py:139-140): the named test needs a file with that name to prove no file is exempt. Nothing in docs/ or README.md names it.

marola-ml removed marola-sea-publish.yml and nothing is self-hosted now, so the exemption
only let a new file of that name reach the runner. No workflow may name self-hosted or an
expression in runs-on/runner; the PR-trigger rule that guarded the exempt file goes with it.
The runners page, the tools table and the README say so.

Closes #168

Tested: workflow_runners.py --self-test red (sea-publish case) then green; devkit-quality.sh rc=0
Co-Authored-By: Claude <noreply@anthropic.com>
@brunogbv
brunogbv force-pushed the chore/168-runners-exemption branch from 33e9c76 to cafaa41 Compare October 17, 2026 11:18
@brunogbv
brunogbv merged commit e2c2ea6 into main Oct 17, 2026
8 checks passed
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.

workflow-runners exempts marola-sea-publish.yml, which no longer exists

1 participant