Skip to content

Bugfix/ruff 016 lint compliant - #351

Merged
LorenzzoQM merged 27 commits into
developfrom
bugfix/ruff_016_lint_compliant
Jul 28, 2026
Merged

LorenzzoQM merged 27 commits into
developfrom
bugfix/ruff_016_lint_compliant

Conversation

@LorenzzoQM

Copy link
Copy Markdown
Contributor

Description

Closes #350

Update files for compatibility with Ruff 0.16.0's new default lint rules, excluding C408.

Type of change

  • 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)

How should this pull request be reviewed?

  • By commit
  • All changes at once

How Has This Been Tested?

Unit and integration tests passed.

Checklist

  • I have performed a self-review of my code
  • I have commented my code in hard-to-understand areas
  • I have made corresponding changes to the documentation and release notes
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Copilot AI review requested due to automatic review settings July 24, 2026 22:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@LorenzzoQM
LorenzzoQM force-pushed the bugfix/ruff_016_lint_compliant branch from 9a57377 to 8b54914 Compare July 24, 2026 23:32

@Mark2000 Mark2000 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Main thought is that ClassVar is stupid, not sure what the point of that rule is. Maybe worth giving the actual types in some places?

Comment thread benchmarks/benchmark.py Resolved
Comment thread benchmarks/aeos.py Resolved
Comment thread docs/source/conf.py Resolved
@LorenzzoQM
LorenzzoQM force-pushed the bugfix/ruff_016_lint_compliant branch from 8b54914 to 0029785 Compare July 27, 2026 14:21
@LorenzzoQM
LorenzzoQM force-pushed the bugfix/ruff_016_lint_compliant branch from 0029785 to af603e0 Compare July 27, 2026 14:38
@LorenzzoQM
LorenzzoQM merged commit aaceb45 into develop Jul 28, 2026
8 checks passed
@LorenzzoQM
LorenzzoQM deleted the bugfix/ruff_016_lint_compliant branch July 28, 2026 19:21
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.

Ruff 0.16.0 causes lint checks to fail on existing code

3 participants