-
Notifications
You must be signed in to change notification settings - Fork 98
implement linting and formatting process #651
Copy link
Copy link
Open
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Refactor 🔧Code level improvements that restructure existing logic without changing behavior.Code level improvements that restructure existing logic without changing behavior.
Description
Metadata
Metadata
Assignees
Labels
Effort > Brief 🐇Small tasks expected to take a few hours up to a couple of days.Small tasks expected to take a few hours up to a couple of days.Impact > Minor 🔷Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Small, backward compatible change. Treat like a patch release (e.g., 0.5.8 → 0.5.9).Refactor 🔧Code level improvements that restructure existing logic without changing behavior.Code level improvements that restructure existing logic without changing behavior.
Description
I would recommend
ruff(https://docs.astral.sh/ruff/). I've been using it for about 2 years and really like it. Very fast, ergonomic, solid tooling (has GitHub Actions). Same creators asuvwhich this project uses.Could use this issue to decide on which rules to enforce, should the idea be accepted.
Do you have any additional supporting notes?
Previously suggested in a comment here: #621 (comment)
I'd be happy to contribute to this implementation. I made a small related change already here: #650