Skip to content

[CI] Add xcodebuild gate for PRs (#33)#37

Merged
Jam-Cai merged 1 commit into
FuJacob:mainfrom
Jam-Cai:ci/build-check-on-prs
Apr 22, 2026
Merged

[CI] Add xcodebuild gate for PRs (#33)#37
Jam-Cai merged 1 commit into
FuJacob:mainfrom
Jam-Cai:ci/build-check-on-prs

Conversation

@Jam-Cai

@Jam-Cai Jam-Cai commented Apr 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

First CI check: xcodebuild build on every PR into main and on pushes to main. Compile errors now fail the PR gate before a reviewer has to look at it.

What's in the workflow

  • Runner: macos-latest
  • Xcode: pinned via setup-xcode@v1 to latest-stable. Explicit bump point — GitHub-hosted images swap their default Xcode without notice and that surfaces as mystery-red or mystery-green builds.
  • Scheme: tabby, Debug config.
  • Signing: CODE_SIGNING_ALLOWED=NO. Runners don't have the dev cert; Automatic signing would otherwise fail the step. Check stays focused on compile correctness.
  • Concurrency: cancels older runs on the same ref so rapid pushes don't pile up stale builds.

What you'll see after merge

  • A Build check on every PR. Green = compiled; red = compile error with xcodebuild output in the run log.
  • Same check runs on main after merge — so a future README badge can reflect current main health.

Follow-ups (separate PRs)

Refs

Refs #33

Adds .github/workflows/build.yml — runs xcodebuild against the tabby
scheme on every PR into main and every push to main. Fails the check
on compile errors so they're caught before review.

- Pinned to `latest-stable` Xcode via setup-xcode; intentional bump
  point when the project moves versions.
- Debug config for speed; same compile surface as Release.
- CODE_SIGNING_ALLOWED=NO because runners lack the dev cert.
- Concurrency guard cancels stale runs on rapid pushes.

Refs #33
@Jam-Cai Jam-Cai merged commit d3ec9d0 into FuJacob:main Apr 22, 2026
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