-
Notifications
You must be signed in to change notification settings - Fork 448
Add more ruff rules #4001
Copy link
Copy link
Open
Labels
Area: InfrastructurePertains to project infrastructure (e.g. CI, linting)Pertains to project infrastructure (e.g. CI, linting)Type: EnhancementEnhancement to existing functionalityEnhancement to existing functionality
Metadata
Metadata
Assignees
Labels
Area: InfrastructurePertains to project infrastructure (e.g. CI, linting)Pertains to project infrastructure (e.g. CI, linting)Type: EnhancementEnhancement to existing functionalityEnhancement to existing functionality
Add more rules to our ruff config. Current ones I'm seeing:
PTHPTPLA brief check shows some things I'd want to look into with the first two. Pylint will need to be more judicious, since a brief look seems to show some things I don't care to fix (overwriting loop variables). That said, I also just saw this while writing:
which is suspicious but somehow isn't a bug, though should be changed anyways:
True