Skip to content

Add shared workflow checks and the workflow convention - #7

Merged
KOliver94 merged 2 commits into
mainfrom
workflow-checks
Oct 19, 2026
Merged

KOliver94 merged 2 commits into
mainfrom
workflow-checks

Conversation

@KOliver94

Copy link
Copy Markdown
Member

Every workflow in the studio's repositories now follows one written convention, docs/workflows.md, and a reusable workflow enforces it.

  • .github/workflows/shared-checks.yml (called from each repository's checks.yml):
    • Workflows runs actionlint, zizmor (with every action required to be pinned by commit SHA) and scripts/check-workflows.mjs over the repository's workflows and composite actions.
    • Dependencies reviews new and changed dependencies for known vulnerabilities and for licences outside the allow-list, in repositories that set dependency-review: true.
  • The checks run from the composite action .github/actions/workflow-checks, which the shared workflow reaches through the $/ self-repository syntax, so tools, configuration and workflow always come from the same commit. actionlint and zizmor run from digest-pinned container images that Renovate keeps current.
  • scripts/check-workflows.mjs covers the parts of the convention no other tool checks: key order, permissions, concurrency, bash as the shell, job names and timeouts, step layout, if conditions, expressions inside run, spacing and quoting. It is dependency-free and tested with node:test.
  • This repository runs the checks on itself through a local reference, so a pull request here tests the version it changes. test.yml follows the convention.

@KOliver94
KOliver94 merged commit 61754fb into main Oct 19, 2026
6 checks passed
@KOliver94
KOliver94 deleted the workflow-checks branch October 19, 2026 21:58
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