Skip to content

Lint Python sources with ruff#85

Merged
mattt merged 13 commits intomainfrom
mattt/ruff
Apr 11, 2023
Merged

Lint Python sources with ruff#85
mattt merged 13 commits intomainfrom
mattt/ruff

Conversation

@mattt
Copy link
Copy Markdown
Contributor

@mattt mattt commented Apr 7, 2023

Depends on #84

This PR adds ruff as a dev dependency to lint Python sources. It enables a handful of checks and fixes resulting violations. Additional checks like ANN for type annotations will be more involved, and can be taken in separate PRs.

@mattt mattt marked this pull request as ready for review April 10, 2023 11:38
@zeke
Copy link
Copy Markdown
Member

zeke commented Apr 11, 2023

Looks good to me, aside from the merge conflict.

mattt added 13 commits April 11, 2023 15:31
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 c892c78 into main Apr 11, 2023
@mattt mattt deleted the mattt/ruff branch April 11, 2023 22:34
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