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.

Sign up for GitHub

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

Jump to bottom

Enable remaining ruff rules #187

Merged

kodiakhq merged 2 commits into sbdchd:main from hamdanal:ruff-all

Sep 2, 2023

hamdanal commented Sep 2, 2023

This is the last PR of a series of PRs updating the project syntax using ruff (continuation of #185).

In this PR I activate the remaining two rules:

other parameter to __eq__ method should always be annotated as object
Quoting forward references (stringifing types) is unnecessary in stubs

This PR is fairly small compared to the previous ones


          Enable remaining ruff rules

3b20c6d

sbdchd added the automerge label

Owner

sbdchd commented Sep 2, 2023

Thank you!


          Merge branch 'main' into ruff-all

ad47347

kodiakhq bot merged commit d1c8fca into sbdchd:main

hamdanal deleted the ruff-all branch

September 2, 2023 20:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels