Skip to content

Conversation

@Epse
Copy link
Member

@Epse Epse commented Oct 10, 2025

Makes the ruff linter very strict, and thus very helpful.
Most rules that are applicable to our codebase have been enabled, with the following exceptions:

  • DTZ: there are exactly two uses of datetime in the codebase, in the comparison of token expiry, naive is fine there
  • D: We acknowledge that our documentation is probably insufficient, but this linting is too aggressive at this time

For the relevant rules, fixes are included

Also adds the lint check to the pre-commit hook, as that had gotten forgotten before.

@Epse Epse merged commit bbb66f8 into main Oct 10, 2025
12 checks passed
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