Skip to content

chore: Add a "validate actions" script to validate workflow/action YAML in CI - W-24800910 - #208

Open
mshanemc wants to merge 1 commit into
mainfrom
sm/W-24800910
Open

mshanemc wants to merge 1 commit into
mainfrom
sm/W-24800910

Conversation

@mshanemc

Copy link
Copy Markdown
Contributor

What issues does this PR fix or reference?

@W-24800910@

What changed

  • Added npm run check:actions to validate workflow and action YAML against the Actions schema.
  • Added checks that inputs passed to local and self-referenced reusable workflows are defined by the called workflow, even when the calling job has if: false.
  • Added PR CI to run the validator and its regression tests, and documented the commands in CONTRIBUTING.md.

Why

Catch invalid reusable-workflow inputs before merge, preventing startup failures like the one caused by passing inputs that ctcOpen.yml and ctcClose.yml no longer define.

@mshanemc
mshanemc marked this pull request as ready for review November 10, 2026 05:42
@mshanemc
mshanemc requested a review from a team as a code owner November 10, 2026 05:42
@mshanemc
mshanemc removed the request for review from a team November 10, 2026 05:42
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