Skip to content

chore(nimbus): Add GHA workflow for Schema tests#14612

Merged
jaredlockhart merged 1 commit intomainfrom
circleci-to-gha
Feb 11, 2026
Merged

chore(nimbus): Add GHA workflow for Schema tests#14612
jaredlockhart merged 1 commit intomainfrom
circleci-to-gha

Conversation

@jaredlockhart
Copy link
Collaborator

Because

  • We are migrating CI from CircleCI to GitHub Actions (EXP-6320)
  • The schemas check job needs to run on PRs that change schemas/

This commit

  • Adds a GitHub Actions workflow that runs make schemas_check on PRs
  • Triggers on changes to schemas/ and the workflow file itself

fixes #14582

Copy link
Contributor

@mikewilli mikewilli left a comment

Choose a reason for hiding this comment

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

One question but otherwise LGTM

@@ -0,0 +1,26 @@
name: Schema Tests

on:
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this also run on pushes to main?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh yeah but I was gonna leave that for when we do the deploy pipeline

Because

* We are migrating CI from CircleCI to GitHub Actions (EXP-6320)
* The schemas check job needs to run on PRs that change schemas/

This commit

* Adds a GitHub Actions workflow that runs make schemas_check on PRs
* Uses the setup-cached-build composite action for GAR-backed Docker
  layer caching
* Adds SCHEMAS_BUILD_FLAGS override point to the Makefile
* Updates the composite action to export SCHEMAS_BUILD_FLAGS

fixes #14582

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jaredlockhart jaredlockhart added this pull request to the merge queue Feb 11, 2026
Merged via the queue into main with commit 7f15d1a Feb 11, 2026
17 checks passed
@jaredlockhart jaredlockhart deleted the circleci-to-gha branch February 11, 2026 20:05
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.

Migrate Check Schemas to GHA

2 participants