Skip to content

ci: consolidate PR workflows into one CI workflow with an all-green gate - #276

Merged
Primajin merged 2 commits into
mainfrom
ci/consolidate-workflows
Aug 20, 2026
Merged

ci: consolidate PR workflows into one CI workflow with an all-green gate#276
Primajin merged 2 commits into
mainfrom
ci/consolidate-workflows

Conversation

@Primajin

Copy link
Copy Markdown
Owner

Summary

Consolidates the three PR-check workflows (jsdoc, lint, test) into a single ci.yml, each preserved as its own job, and adds an All checks green aggregator so a single required status can gate the PR.

Structure mirrors the validated reference template (Primajin/Gyros ci.yml):

  • One workflow on push: [main] + pull_request: [opened, reopened, synchronize, closed].
  • Each former workflow becomes a job carrying its own permissions: (copied from the origin workflow's top-level permissions; contents: read where the origin had none).
  • Each consolidated job keeps its original push+PR trigger intent via if: github.event_name == 'push' || github.event.action != 'closed'.
  • A final all-green job named exactly All checks green (if: always(), needs: every other job) fails if any job failed or was cancelled.
  • Node bumped to '24' to match the reference idiom.

Left separate (untouched)

  • codeql-analysis.yml
  • release-please.yml

Removed

  • .github/workflows/jsdoc.yml
  • .github/workflows/lint.yml
  • .github/workflows/test.yml

Notes

  • The gh-pages concurrency group from jsdoc.yml is preserved at the workflow top level (matching the reference).
  • lint retains its original permissions: write-all; jsdoc and test get contents: read.

🤖 Generated with Claude Code

Merge the jsdoc, lint, and test workflows into a single ci.yml, each
preserved as its own job with its own permissions, and add an
"All checks green" aggregator job that fails if any job failed or was
cancelled. codeql-analysis.yml and release-please.yml are left untouched.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Primajin

Primajin commented Aug 20, 2026

Copy link
Copy Markdown
Owner Author

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (8db15bd) to head (773fd65).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #276   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            2         2           
  Lines           95        95           
=========================================
  Hits            95        95           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Keep the rest of CI out of the gh-pages concurrency group so it is not
serialised/cancelled alongside the docs deploy.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Primajin
Primajin merged commit a25c61b into main Aug 20, 2026
12 checks passed
@Primajin
Primajin deleted the ci/consolidate-workflows branch August 20, 2026 12:35
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