-
Notifications
You must be signed in to change notification settings - Fork 0
chore: enable ruff rules disabled during template sync #6
Copy link
Copy link
Open
Labels
choreMaintenance, tooling, CI tasksMaintenance, tooling, CI tasks
Description
Chore Type
Tooling improvements
Description
During the pyproject-template sync (#2, PR #5), several ruff rules were added to the ignore list to allow existing code to pass. These should be addressed to fully adopt the template's stricter linting standards.
Rules to enable
| Rule | Description | Est. Errors | Priority |
|---|---|---|---|
ANN |
Type annotations (flake8-annotations) | ~1349 | High |
N |
Naming conventions (pep8-naming) | ~20 | Medium |
E501 |
Line too long (>100 chars) | ~162 | Low |
SIM108 |
Use ternary operator | ~4 | Low |
B019 |
lru_cache on methods | ~3 | Low |
Proposed Changes
- Enable one rule at a time
- Fix errors or add targeted per-file-ignores
- Create separate PRs for each rule
Success Criteria
-
ANNrule enabled (or per-file-ignores added) -
Nrule enabled -
E501rule enabled -
SIM108rule enabled -
B019addressed (fix or document why ignored) -
doit checkpasses with all rules enabled
Additional Context
- Original bastproxy ruff rules preserved in issue chore: synchronize with latest pyproject-template #2 comments
- Template sync PR: chore: sync tooling and doit commands with pyproject-template #5
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
choreMaintenance, tooling, CI tasksMaintenance, tooling, CI tasks