Skip to content

Unified CI workflow for smarter PR gating#148

Merged
alan-george-lk merged 38 commits into
mainfrom
feature/unified_ci
Jun 3, 2026
Merged

Unified CI workflow for smarter PR gating#148
alan-george-lk merged 38 commits into
mainfrom
feature/unified_ci

Conversation

@alan-george-lk
Copy link
Copy Markdown
Collaborator

@alan-george-lk alan-george-lk commented Jun 1, 2026

  • Unifies all workflows under a top-level ci.yml workflow
    • This helps better visualize the entire workflow in a single link, all step summaries are viewable on a single page (e.g. clang-* results), and reduces run entries in the Actions repo tab
    • Also allows repo settings to reference one check for passing/failing PRs, and let's that workflow correctly dispatch which things to actually run
  • Removes CPP_EXAMPLE_COLLECTION ref (duplicated across two files) and simply reads the submodule hash via git
  • Attempt to use custom cache since sccache doesn't appear to be working
  • Move out clang-* tools to cpp-checks.yml in case we want to move it into a separate repo for use across C++ projects
  • Remove pin-check.yml, now natively supported in repo settings (turned on now)

@alan-george-lk alan-george-lk marked this pull request as ready for review June 3, 2026 18:25
Copy link
Copy Markdown
Collaborator

@stephen-derosa stephen-derosa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a lot of magic going on, but if it fits it ships.

Consider drafting a pre-release off of this branch and explicitly targeting those artifacts in a manual cpp-example-collection build locally. Or at least the "Build against a local SDK" stage here

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/docker-images.yml
Comment thread .github/workflows/docker-validate.yml
Comment thread AGENTS.md
@alan-george-lk alan-george-lk merged commit f65cc98 into main Jun 3, 2026
25 checks passed
@alan-george-lk alan-george-lk deleted the feature/unified_ci branch June 3, 2026 23:45
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.

2 participants