Skip to content

fix(NOJIRA): slim ci-standard-checks workflow - #284

Draft
adriancarayol wants to merge 3 commits into
mainfrom
ci-std-checks-slim-final
Draft

adriancarayol wants to merge 3 commits into
mainfrom
ci-std-checks-slim-final

Conversation

@adriancarayol

Copy link
Copy Markdown

What

Slims the shared ci-standard-checks-workflow.yaml (one file, +12 lines):

  • Job-level concurrency per caller workflow and PR; only pull_request runs are cancelled, pushes/merge_group are never cancelled.
  • Skip Dependabot PRs at job level (PR author). A subset of what the action already skips, so no scan coverage is lost.
  • Run setup-node only when an openapi.yaml exists within depth 3 (still needed for validate-openapi).
  • Checkout depth unchanged on purpose.

Status

Draft, not ready to merge. Not yet verified on a real runner; needs a pilot caller pointed at this branch (cancel behaviour on required checks, skipped job on a required check, openapi path). Some callers reference @main, so merging changes them immediately; pilot before merge.

🤖 Generated with Claude Code

adriancarayol and others added 3 commits November 13, 2026 15:48
…ot PRs, set up node only for openapi repos

- concurrency: a newer event on the same PR cancels the run in progress.
  push and merge_group runs use their run id as group, so they are never
  cancelled or queued. The group prefix differs from every group the
  callers set today, because an equal group cancels the caller's own run.
- if: Dependabot pull requests no longer take a runner. The action
  already skips all checks for them (triggeredByBot), and a skipped job
  satisfies a required check.
- setup-node now runs only when validate-openapi can need npx, i.e. when
  an openapi.yaml exists within the depth the script searches. It cannot
  be dropped outright: the runner image has no node on PATH.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…equest only

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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