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
Add new rules to ruff config #4250
Conversation
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.
interesting updates!
@@ -165,7 +165,7 @@ class SimpleUpdateProcessor(BaseUpdateProcessor): | |||
|
|||
async def do_process_update( | |||
self, | |||
update: object, | |||
update: object, # noqa: ARG002 |
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.
tbh this seems like a bug in ARG002 - is the parent class defines this, ruff shouldn't complain …
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'll open an issue on their repo and see what they think
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 still be okay with merging btw
apparently pre-commit does not run on rebasing
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.
LEGTM, then :) Let me know if I should merge or if you have additional changes planned
all done from my side. Can be merged |
Most notably adds the "ARG" rule which I deliberately didn't add previously. #4180 (comment) prompted this.