Skip to content

fix: resolve CI workflow parse error for fixture checks#15

Closed
rlippmann wants to merge 5 commits intomainfrom
codex/fixture-rename
Closed

fix: resolve CI workflow parse error for fixture checks#15
rlippmann wants to merge 5 commits intomainfrom
codex/fixture-rename

Conversation

@rlippmann
Copy link
Copy Markdown
Owner

@rlippmann rlippmann commented Apr 24, 2026

Summary

  • Replace unsupported expression-based temp paths in CI env with explicit /tmp paths.
  • Preserve fixture drift validation against pinned Python source-of-truth fixtures.

Why this change

  • GitHub Actions was rejecting the workflow at parse time (Unrecognized named-value: runner).
  • With the parse issue fixed, CI runs again and required status checks can enforce merge gating.

Details

  • PY_FIXTURE_CHECKOUT now uses /tmp/context-compiler-source.
  • FIXTURES_SOURCE in the fixture drift step now uses /tmp/context-compiler-source/tests/fixtures/conformance.
  • No fixture JSON or engine semantics changes in this PR.

@rlippmann rlippmann changed the title fix: use explicit temp path in ci fixture checkout fix: resolve CI workflow parse error for fixture checks Apr 24, 2026
@rlippmann
Copy link
Copy Markdown
Owner Author

Closing in favor of #16, which carries the same CI parse fix on a clean single-commit branch based on current main.

@rlippmann rlippmann closed this Apr 24, 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