Skip to content

Type check Python source code with ruff and mypy #87

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
Apr 11, 2023
Merged

Conversation

mattt
Copy link
Contributor

@mattt mattt commented Apr 11, 2023

Depends on #85

This PR adds the ANN rules in ruff and enables mypy for type checking Python sources. It also fixes the new warnings and errors by adding missing type annotations and fixing existing annotations that weren't correct.

@mattt mattt force-pushed the mattt/ruff-ann branch 3 times, most recently from a668548 to deb92e0 Compare April 11, 2023 20:50
@zeke
Copy link
Member

zeke commented Apr 11, 2023

Nice. 👍🏼

mattt added 15 commits April 11, 2023 15:34
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
Signed-off-by: Mattt Zmuda <mattt@replicate.com>
@mattt mattt merged commit afe0092 into main Apr 11, 2023
@mattt mattt deleted the mattt/ruff-ann branch April 11, 2023 22:38
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.

2 participants