-
Notifications
You must be signed in to change notification settings - Fork 243
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
Replace black/pylint with ruff #1102
Comments
Hi @qarmin, Thank you for the suggestion. I've been reading Ruff's documentation and so far ruff is not a drop-in replacement for pylint I am running some tests, and maybe we can use ruff formatter replacing black and use the ruff linter alongside pylint. |
Yes, ruff is not a 1:1 replacement for pylint. Ruff + mypy would probably offer almost everything that pylint offers |
Currently static checks takes 1 minute to apply - https://github.com/py-pdf/fpdf2/actions/runs/7676162269/job/20923341563
With ruff, this time should decrease to max ~10s without cache and 1s with cache
The text was updated successfully, but these errors were encountered: