Skip to content

ci: lint GitHub Actions workflows with actionlint - #32

Open
jrepp wants to merge 2 commits into
mainfrom
release/npm-oidc-publish
Open

ci: lint GitHub Actions workflows with actionlint#32
jrepp wants to merge 2 commits into
mainfrom
release/npm-oidc-publish

Conversation

@jrepp

@jrepp jrepp commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

Add an actionlint job to CI that validates every GitHub Actions workflow (ci.yml, pages.yml, release.yml) on every push and pull request.

Fixes a gap where workflow changes (e.g. release.yml, which only triggers on tag pushes/workflow_dispatch) were never exercised or linted in PR CI.

Changes

  • New workflow-lint job in .github/workflows/ci.yml
  • Downloads pinned actionlint v1.7.12 and verifies its sha256 checksum
  • Runs ./actionlint over the repository workflows
  • Fixes pre-existing issues actionlint surfaced in release.yml:
    • Quote $GITHUB_ENV/$GITHUB_OUTPUT (SC2086)
    • Use ./* glob in the WASM zip step (SC2035)
    • Bump softprops/action-gh-release to v3.0.3 (v1 runner too old for GitHub Actions)

Validation

  • actionlint v1.7.12 checksum matches the upstream actionlint_1.7.12_checksums.txt
  • workflow-lint job passes on this PR; all other CI checks green

@jrepp
jrepp force-pushed the release/npm-oidc-publish branch from 86f7bf7 to 8b3da6f Compare September 9, 2026 09:54
@jrepp jrepp changed the title ci(release): publish npm via trusted publishing (staged) ci: lint GitHub Actions workflows with actionlint Sep 9, 2026
Validate ci.yml, pages.yml, and release.yml on every push and pull
request. Downloads the pinned actionlint v1.7.12 binary and verifies its
sha256 checksum.
Resolve issues surfaced by actionlint in release.yml:
- Quote $GITHUB_ENV and $GITHUB_OUTPUT (SC2086)
- Use ./* glob for zip (SC2035)
- Bump softprops/action-gh-release to v3.0.3 (v1 runner too old)
@jrepp
jrepp force-pushed the release/npm-oidc-publish branch from 560cd99 to 95639c3 Compare September 9, 2026 10:08
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