-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
(RUF012) Fixed mutable class Defaults- Task 3 #10284
(RUF012) Fixed mutable class Defaults- Task 3 #10284
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.
Looks great to me but I'll need to get a double check from staff :)
Hi @RayBB , I made the changes according to your guidance. If there's anything that needs to be modified, please guide me. |
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.
Just one question but otherwise this is on the right path!
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.
Great work on this one. I think we're ready for @cdrini to take a look.
Testing:
- Adding books to bookshelf worked
- Search autocomplete worked
@drona-gyawali, it may be the case the the tests will pass after rebasing. It may also be the case that it won't help. However, it's likely worth a shot. See Updating your Pull Request if you need any help with that. |
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.
Awesome thank you @drona-gyawali ! Looks good, just a point of feedback around the tuple type hints.
for more information, see https://pre-commit.ci
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.
lgtm! Thank you @drona-gyawali !
Part of #10196
NOTE: These are NOT automated changes, they are done by hand
This PR addresses Part of RUF012 Rule - Task 3, implementing
typing.ClassVar
across 4 files with a total of 13 changes. The updates primarily focus on enhancing code quality and resolving linting issues (RUF012) related to mutable class attributes.Technical
4 files
to use typing.ClassVar for clarity and correctness.Testing
Ran ruff check --select RUF012 to confirm no further warnings.
Screenshot
Stakeholders
@RayBB