Skip to content

refactor: harden ruff rules#174

Merged
mdevolde merged 1 commit into
jxmorris12:masterfrom
mdevolde:refactor/ruff
Jun 8, 2026
Merged

refactor: harden ruff rules#174
mdevolde merged 1 commit into
jxmorris12:masterfrom
mdevolde:refactor/ruff

Conversation

@mdevolde

@mdevolde mdevolde commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

refactor: harden ruff rules

Why the pull request was made

To harden the configuration of ruff and ensure that the code base is idiomatic.

Summary of changes

  • Harden ruff config
  • Patch new warnings from new ruff warns
  • Fix a little doc error in the README
  • Update mypy and ruff version
  • uv lock upgrade
  • Edited mypy conf in pyproject.toml to have a similar config than before but more explicit

Screenshots (if appropriate):

Not applicable.

How has this been tested?

Applied local tests.

Resources

https://docs.astral.sh/ruff/rules/

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (changes to documentation only)
  • Refactor / code style update (non-breaking change that improves code structure or readability)
  • Tests / CI improvement (adding or updating tests or CI configuration only)
  • Other (please describe):

Checklist

  • Followed the project's contributing guidelines.
  • Updated any relevant tests.
  • Updated any relevant documentation.
  • Added comments to your code where necessary.
  • Formatted your code, run the linters, checked types and tests.

@mdevolde mdevolde self-assigned this Jun 8, 2026
@mdevolde mdevolde force-pushed the refactor/ruff branch 2 times, most recently from 08026cf to 79181b3 Compare June 8, 2026 11:14
@mdevolde mdevolde merged commit 7a7bcb3 into jxmorris12:master Jun 8, 2026
8 checks passed
@mdevolde mdevolde deleted the refactor/ruff branch June 8, 2026 11:31
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