You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Unify formatting, linting, and continuous integration.
There should only be one way of linting the code, not two (pre-commit and tox).
The likely solution would be to move all formatting and linting to the pre-commit hook and to call pre-commit run --all-files in the GitHub Action. It might also be time to move to Ruff instead of pylint+black+isort. Remember to update the developer guidelines.
The text was updated successfully, but these errors were encountered:
pnkraemer
changed the title
Unify formatting, linting, and continuous integration by bundling up black, isort, pylint, etc., with the pre-commit hook. The likely solution would be to move all formatting and linting to the pre-commit hook and to call pre-commit run --all-files in the GitHub Action. It might also be time to move to Ruff instead of pylint+black+isort. Remember to update the developer guidelines.
Unify formatting, linting, and continuous integration by bundling up black, isort, pylint, etc., with the pre-commit hook
Jan 26, 2024
pnkraemer
changed the title
Unify formatting, linting, and continuous integration by bundling up black, isort, pylint, etc., with the pre-commit hook
Unify formatting, linting, and continuous integration by bundling up the linting-requirements in tox with the pre-commit hook
Jan 26, 2024
pnkraemer
changed the title
Unify formatting, linting, and continuous integration by bundling up the linting-requirements in tox with the pre-commit hook
Bundle up the linting-requirements in tox with the pre-commit hook
Jan 26, 2024
Unify formatting, linting, and continuous integration.
There should only be one way of linting the code, not two (pre-commit and tox).
The likely solution would be to move all formatting and linting to the pre-commit hook and to call
pre-commit run --all-files
in the GitHub Action. It might also be time to move to Ruff instead of pylint+black+isort. Remember to update the developer guidelines.The text was updated successfully, but these errors were encountered: