Skip to content

feat(gh-2110): setup for astral-sh/ty#2111

Merged
juanitorduz merged 3 commits intopyro-ppl:masterfrom
Qazalbash:try-ty
Dec 21, 2025
Merged

feat(gh-2110): setup for astral-sh/ty#2111
juanitorduz merged 3 commits intopyro-ppl:masterfrom
Qazalbash:try-ty

Conversation

@Qazalbash
Copy link
Collaborator

This PR introduces faster type checking by astral-sh/ty as an alternative to mypy and fixes #2110.

I have not removed mypy entirely; it is still the primary type checker. In the future, we will transfer small type-checking workloads to ty and move the dependence from mypy. For the very same reason, I have provided the --exit-zero flag to force exit with 0 exit code so that it does not interfere with our CI.

Copy link
Collaborator

@juanitorduz juanitorduz left a comment

Choose a reason for hiding this comment

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

QQ: Why do you want to keep mypy now? What is preventing us from the whole migration? (or is it just because ins in beta mode?)

@fehiepsi
Copy link
Member

Looks great to me pending @juanitorduz's comments.

@Qazalbash
Copy link
Collaborator Author

QQ: Why do you want to keep mypy now? What is preventing us from the whole migration? (or is it just because ins in beta mode?)

Yes, the reason is the beta mode, and also, we are not sure how much we can utilize it.

@juanitorduz
Copy link
Collaborator

ok! makes sense, then let's keep #2110 open it would be great to just rely on ty :)

@juanitorduz juanitorduz merged commit 4283a50 into pyro-ppl:master Dec 21, 2025
9 checks passed
@Qazalbash Qazalbash deleted the try-ty branch December 21, 2025 08:52
@Qazalbash Qazalbash mentioned this pull request Dec 22, 2025
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.

Adopt astral-sh/ty for Faster Static Type Checking

3 participants