Skip to content

CI: run the tests, the Swift build and the smoke check on every pull request - #1

Merged
dancechessmaster merged 1 commit into
mainfrom
ci/pr-checks
Sep 7, 2026
Merged

CI: run the tests, the Swift build and the smoke check on every pull request#1
dancechessmaster merged 1 commit into
mainfrom
ci/pr-checks

Conversation

@dancechessmaster

Copy link
Copy Markdown
Contributor

Until now the only workflow ran on a tag push, so a pull request carried no checks at all.

This runs what a release would run, minus the DMG — cargo test, bindings regenerated by build-core.sh, the SwiftPM build, and StudioSmoke against a real Stockfish — on the same macos-15 runner, so a green check here means the release build gets through too.

The structural work that follows (per-window databases, native tabs) touches the write-back path and is going in as pull requests precisely so it can be reviewed and reverted as a unit. A check that means something is the first half of that.

🤖 Generated with Claude Code

https://claude.ai/code/session_01F9o9DSzZas4GsekCpG5M8T

Until now the only workflow ran on a tag push, so a pull request carried
no checks at all. This runs what a release would run, minus the DMG:
cargo test, the bindings regenerated by build-core.sh, the SwiftPM build,
and StudioSmoke against a real Stockfish — on the same macos-15 runner, so
a green check here means the release build gets through too.

The structural work that follows (per-window databases, native tabs)
touches the write-back path, and it is going in as pull requests
precisely so it can be reviewed and reverted as a unit. A check that
means something is the first half of that.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01F9o9DSzZas4GsekCpG5M8T
@dancechessmaster
dancechessmaster merged commit b99084a into main Sep 7, 2026
1 check passed
@dancechessmaster
dancechessmaster deleted the ci/pr-checks branch September 7, 2026 06:01
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