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.

Sign up for GitHub

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

Jump to bottom

Use Ruff instead of other lint/formatting tools #59

Open

mwtoews wants to merge 1 commit into csdms:master from mwtoews:use-ruff

Contributor

mwtoews commented Dec 2, 2024

This PR uses Ruff to replace several lint and formatting tools. Ruff is modern and very fast.

Replace Black with the Ruff Formatter, which incurs a minor change to whitespace.
Replace flake8 with several Ruff rules.
Replace pyupgrade with UP rule.
Replace reorder-python-imports with Ruff I (isort) and FA (flake8-future-annotations) rules. This change modified some of the code, combining imports from a single module into one line.
Replace pydocstyle with D rule.

Other changes:

Set the pre-commit.ci configuration to update quarterly, i.e. less frequently
Remove nb-ensure-clean pre-commit, since there are no *.ipynb files in this repo
Some of the options for the mypy pre-commit needed to be commented-out to work (were these needed?)


          Use Ruff instead of other lint/formatting tools

coveralls commented Dec 2, 2024

coverage: 78.319% (-0.2%) from 78.509%
when pulling 5939971 on mwtoews:use-ruff
into b0ef8b6 on csdms:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet