-
-
Notifications
You must be signed in to change notification settings - Fork 213
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
Use ruff for pyupgrade and isort #926
Use ruff for pyupgrade and isort #926
Conversation
@@ -379,7 +378,7 @@ def from_dict(conversion: dict[str, Any]) -> v4b.ChannelConversion: | |||
return conversion | |||
|
|||
|
|||
def to_dict(conversion: ChannelConversionType) -> Union[dict, None]: | |||
def to_dict(conversion: ChannelConversionType) -> dict | None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't this require Python >= 3.10 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it does, but ruff ignores it, because the file contains from __future__ import annotations
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, let me know when it is ready for merging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd say it's ready. This little change resulted in 73 modified files, that's enough for one PR 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small changes ,😁
17c7a8d
into
danielhrisca:development
Thanks! |
You could activate many more rules to be checked on commit and in CI.