Skip to content

chore: pin ty check execution#69

Merged
vienneraphael merged 2 commits into
mainfrom
chore/pin-ty-check
Jun 13, 2026
Merged

chore: pin ty check execution#69
vienneraphael merged 2 commits into
mainfrom
chore/pin-ty-check

Conversation

@vienneraphael

Copy link
Copy Markdown
Owner

Summary

  • add ty to the project dev dependency flow so it is resolved through uv.lock
  • run the local ty-check hook via uv run --locked ty check instead of floating uvx ty check
  • mark the current intentional test diagnostics with rule-specific ty ignores

Why

CI was running ty through uvx, which fetches the current published package rather than the project lockfile. That let type-checker drift break unrelated PR CI. This makes the hook deterministic and clears the diagnostics currently present on main.

Verification

  • uv run --locked ty check --python=.venv/ --exclude=examples/
  • uv run --locked pytest tests/test_api.py tests/test_core.py
  • uv run --locked prek run -a
  • $code-review review loop: no findings

@vienneraphael vienneraphael marked this pull request as ready for review June 13, 2026 00:57
@vienneraphael vienneraphael merged commit bb28445 into main Jun 13, 2026
5 checks passed
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