Skip to content

Add ty as another type checker - #212

Merged
sbdchd merged 4 commits into
sbdchd:mainfrom
oliverhaas:add-ty
Mar 11, 2026
Merged

Add ty as another type checker#212
sbdchd merged 4 commits into
sbdchd:mainfrom
oliverhaas:add-ty

Conversation

@oliverhaas

@oliverhaas oliverhaas commented Jan 12, 2026

Copy link
Copy Markdown
Contributor

Given the success of Astral's uv I expect their new type checker "ty" to quickly become a staple in the python ecosystem, so I suggest adding it to this project.

@oliverhaas
oliverhaas marked this pull request as ready for review January 12, 2026 11:17

@sbdchd sbdchd left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you!

@sbdchd sbdchd added the automerge auto merge PRs using kodiak label Jan 12, 2026
@sbdchd
sbdchd merged commit 07ef590 into sbdchd:main Mar 11, 2026
2 checks passed
@oliverhaas
oliverhaas deleted the add-ty branch March 11, 2026 15:36
kodiakhq Bot pushed a commit that referenced this pull request Jul 29, 2026
While turning around the [last PR](#227), I stubbed my toe on `ty` checks running in CI but not in pre-commit.  [PR #212](#212) looks like it added the check, but not the local devtools or mention in the readme. This PR adds the ty type checker to the README, linting script, and pre-commit hooks, matching the requirements of the ci gate.

I had this in my branches, but am a little ambivalent about further cementing `ty` as a supported type checker, because I ran into a confusing bug while using it.  I compromised [by writing up the problem](#228), and being embarrassed about the wall of text.  Can't win I guess.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge auto merge PRs using kodiak

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants