Skip to content

ci: run the lossless-repair proof in CI (install flac on Ubuntu + macOS)#35

Merged
Guillain-RDCDE merged 1 commit into
mainfrom
ci/flac-in-test-job
Jun 3, 2026
Merged

ci: run the lossless-repair proof in CI (install flac on Ubuntu + macOS)#35
Guillain-RDCDE merged 1 commit into
mainfrom
ci/flac-in-test-job

Conversation

@Guillain-RDCDE
Copy link
Copy Markdown
Owner

Installe le binaire flac dans le job de test (Ubuntu via apt, macOS via brew) pour que la preuve de réparation lossless (tests/test_repair_lossless.py) tourne en CV au lieu de se skipper. Windows continue de skipper proprement (le test s'auto-skip, pas de dépendance Chocolatey ajoutée).

Ferme le fil ouvert dans le PR #34 : la garantie « la réparation rend un audio bit-pour-bit identique » est désormais vérifiée à chaque push.

🤖 Generated with Claude Code

The end-to-end lossless-repair proof (tests/test_repair_lossless.py) module-skips
when the `flac` binary is absent. CI runners did not have it, so the guarantee
that repair returns bit-identical audio was never actually exercised in CI.

Add `flac` to the Ubuntu (apt) and macOS (brew) system-dependency steps so the
proof runs on every push on those platforms. Windows keeps skipping gracefully
(the test self-skips; no Chocolatey dependency added).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Guillain-RDCDE Guillain-RDCDE merged commit f4ce4c0 into main Jun 3, 2026
14 checks passed
@Guillain-RDCDE Guillain-RDCDE deleted the ci/flac-in-test-job branch June 3, 2026 11:50
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