Skip to content

fix(release): validate candidate workflows and next push checks - #124

Merged
roodboi merged 1 commit into
nextfrom
codex/prerelease-workflow-validation
Oct 3, 2026
Merged

roodboi merged 1 commit into
nextfrom
codex/prerelease-workflow-validation

Conversation

@roodboi

@roodboi roodboi commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

GitHub rejected the merged prerelease workflow before any jobs ran: the stable tag glob treated + as a quantifier, and the reusable workflow referenced runner.temp where the runner context is unavailable. Escape the literal plus and select temporary build/publication paths through GITHUB_ENV on the assigned runner. Publication protection, exact-source checks and stable release isolation remain unchanged.

CI previously defined only a tag filter, so branch pushes did not schedule the exact next push checks required for publication. Explicitly admit main and next pushes while keeping pull-request CI and release-tag exclusion. Add pinned actionlint validation to the existing required model job, with the repository's Blacksmith runner labels declared, so syntax/context errors are caught before merge.

Validation: actionlint 1.7.12 reproduced all four GitHub validation errors on the original files and passes on the corrected CI/release/reusable workflow files. All 13 focused prerelease controls pass, including branch-push admission; typecheck, quality/privacy checks, changed-test lint and diff checks pass. Hosted required checks must pass on this exact head before merge. No tag or release was created by this fix.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@roodboi
roodboi merged commit 903f404 into next Oct 3, 2026
10 checks passed
@roodboi
roodboi deleted the codex/prerelease-workflow-validation branch October 3, 2026 03:49
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