Skip to content

ci: add aggregate required check - #28

Open
chaptersix wants to merge 1 commit into
dependency-rollupfrom
ci-passed-gate
Open

ci: add aggregate required check#28
chaptersix wants to merge 1 commit into
dependency-rollupfrom
ci-passed-gate

Conversation

@chaptersix

@chaptersix chaptersix commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add the stable CI Passed aggregate job to Validate
  • require the reusable verification workflow to succeed; it covers bundle freshness, linting, formatting, and the Linux/macOS/Windows test matrix
  • make the aggregate job fail when the reusable verification workflow fails, is cancelled, or is skipped

After this stack merges, configure CI Passed as the required status check for main. Future changes can adjust the reusable workflow’s checks without changing branch protection.

Validation

  • YAML parsing for .github/workflows/validate.yml
  • local success and failure predicate checks
  • GitHub Actions: CI Passed green with the reusable verification workflow
  • git diff --check

Stack

Base: #27 (dependency-rollup). Merge after #27; GitHub will retarget this PR to main.

@chaptersix
chaptersix marked this pull request as ready for review August 13, 2026 22:56
@chaptersix
chaptersix requested a review from a team as a code owner August 13, 2026 23:19
@chaptersix
chaptersix force-pushed the ci-passed-gate branch 2 times, most recently from 9077893 to 08e5b91 Compare August 13, 2026 23:23
@chaptersix

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: fd9453670a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

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