Repository navigation
Fix CI failures on Linux and clean checkouts - #18
Merged
Merged
Conversation
The fixture bootstrap refuses to run in the real checkout, so the jobs now use an isolated workspace. Test setup copies the schemas instead of linking them, because Linux followed the link and copied the folder onto itself. Also fixes what that failure was hiding: f-strings that only parse on Python 3.12, a Windows drive path accepted as a package path on Linux, tests that relied on directory order or earlier tests, and FastAPI 0.137 turning app.routes into a tree. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Summary
audit,validateandtestall fail onmain. They pass on Windows dev machines because the causes only show up on Linux or on a clean checkout.audit,validate,test: the jobs now setALEPHTAV_ROOT_DIR, because the fixture bootstrap refuses to run in the real checkout. The audit artifact path follows the workspace.test, 352 errors:tests/conftest.pylinkedschemas/into the test workspace, so on Linux the bootstrap copied the folder onto itself. It now copies, as Windows already did.validate: 9 multi-line f-strings only parse on Python 3.12+, and the project targets 3.11.workspace_path:C:/escapewas accepted as a package path on Linux. This is a product fix.<0.137, whereapp.routesbecame a tree. The route coverage tests read it as a flat list, so the MCP coverage guard would pass while checking nothing.Test plan
audit,validateandtestpass on this PRnpm run buildOK,ruff --select Fclean🤖 Generated with Claude Code