pymc-devs / pymc-devs/pytensor
Consider replacing flake8/pylint by ruff in pre-commit
Open
Nobody has claimed this yet.
GitHub CI/CD
pre-commit
- Dominant language
- Python
- Stars
- 644
- Forks
- 208
- Avg merge
- 2d 14h
- Merged PRs (30d)
- 16
Description
Description
Besides being way way faster, it would allow us to get rid of setup.cfg which AFAICT exists only because flake8 does not support pyproject.toml: https://github.com/PyCQA/flake8/issues/234.
See #295
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the pre-commit configuration and setup.cfg, then read the related discussion in #295. Determine how flake8 and pylint are currently invoked and whether setup.cfg is used only for them. Done means the intended checks use ruff, setup.cfg can be removed if no longer needed, and the existing checks pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100