Skip to content

Conversation

@jbcoe
Copy link
Owner

@jbcoe jbcoe commented Jan 31, 2026

closes #9

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR replaces mypy with ty as the project's type checker and updates dependency management and CI to match.

Changes:

  • Update development dependencies in pyproject.toml and uv.lock to remove mypy (and its supporting packages) and add ty.
  • Configure ty in pyproject.toml with an override for py_cppmodel.py to ignore unresolved-attribute diagnostics.
  • Update local test script and GitHub Actions workflow to run uv run ty check . instead of mypy, and simplify the CI Python setup step.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
uv.lock Regenerates the lockfile to remove mypy, mypy-extensions, librt, and pathspec, and add ty along with updated dev dependency metadata for the project.
pyproject.toml Switches the dev type-checking dependency from mypy to ty and adds a tool.ty override for py_cppmodel.py to ignore unresolved attribute diagnostics.
test.macos.sh Updates the local test script to run uv run ty check . instead of uv run mypy *.py --check-untyped-defs.
.github/workflows/python.yml Adjusts the CI workflow to use ty for type checking and relies on setup-uv for Python installation without an extra uv python install step.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jbcoe jbcoe merged commit b6b8624 into main Jan 31, 2026
5 checks passed
@jbcoe jbcoe deleted the jbcoe/ty-for-type-checks branch January 31, 2026 18:37
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.

Use TY for type checking

2 participants