Skip to content

Apply ruff/flake8-simplify rules (SIM) #349

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

Closed
wants to merge 3 commits into from

Conversation

DimitriPapadopoulos
Copy link
Contributor

No description provided.

SIM109 Use `in` instead of multiple equality comparisons
SIM118 Use `key in dict` instead of `key in dict.keys()`
SIM212 Use `if` instead of `if not`
@astanin
Copy link
Owner

astanin commented Jul 23, 2025

Purely formatting changes increase the risk of merge conflicts with other contributors.
I don't want to accept them unless there is concrete benefit.
The project is already using black and flake8 for automatic formatting and linting.

@astanin astanin closed this Jul 23, 2025
@DimitriPapadopoulos
Copy link
Contributor Author

These are not "purely formatting changes".

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

Successfully merging this pull request may close these issues.

2 participants