Skip to content
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

Use ruff for linting and formatting #44

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

Dunedan
Copy link
Collaborator

@Dunedan Dunedan commented May 25, 2024

This replaces linting with prospector and bandit with ruff, which offers way more checks, is more actively developed and is much faster. This commit also adds a pre-commit hook to check for properly formatted code using ruff as well.

While this commit touches most of the code, these are all non-functional changes and only fix linting problems and properly format the code.

This replaces linting with `prospector` and `bandit` with `ruff`, which
offers way more checks, is more actively developed and is _much_ faster.
This commit also adds a pre-commit hook to check for properly formatted
code using ruff as well.

While this commit touches most of the code, these are all non-functional
changes and only fix linting problems and properly format the code.
@Dunedan Dunedan merged commit 64d6cf2 into 0ad:master Jun 4, 2024
3 checks passed
@Dunedan Dunedan deleted the use-ruff branch June 4, 2024 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant