Repository navigation
Add CI: run the Linux-portable ContinuityCore tests on every PR - #78
Merged
sanylax0 merged 1 commit intoJul 18, 2026
Merged
Conversation
The repo had no CI despite ~127 unit tests. Following the AGENTS.md recipe, assemble a throwaway package from the non-Accelerate sources and run swift test on Linux — 111 of the cases (parsers, Camelot, flow ordering, curves, transition plan, loudness, silence trimming) run on every PR; the Accelerate-backed BPM/key suites still need a Mac.
sanylax2
marked this pull request as ready for review
July 18, 2026 16:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
The repo has ~127 ContinuityCore unit tests and no CI at all — nothing runs them on PRs. AGENTS.md already documents the exact Linux-compatible recipe; this turns it into a GitHub Actions workflow.
How
.github/workflows/core-tests.yml(PRs + pushes to main), running in theswift:6.0container: it assembles a throwaway SwiftPM package from the ContinuityCore sources/tests minus the Accelerate-dependent files (BeatTracker/KeyDetectorand their three test files), thenswift test. That's 111 of the test cases — Camelot/flow ordering, crossfade curves, transition plan, loudness, silence trimming, YouTube/Spotify URL and page parsing — on every PR. The BPM/key suites still require a Mac, as documented.Testing
The assembly recipe mirrors AGENTS.md's documented (and previously exercised) subset; the first run on this PR is itself the validation.
🤖 Generated with Claude Code
https://claude.ai/code/session_013TJoWkqg8bzkGdzxWhWjWP
Generated by Claude Code