Skip to content

cleanup: update ruff and fix all ruff complaints - #383

Merged
fuchsi-huber merged 3 commits into
fix/timestamp-in-testsfrom
cleanup/ruff
Sep 2, 2026
Merged

fuchsi-huber merged 3 commits into
fix/timestamp-in-testsfrom
cleanup/ruff

Conversation

@fuchsi-huber

@fuchsi-huber fuchsi-huber commented Sep 1, 2026 •

Copy link
Copy Markdown
Contributor

Pull request

Please confirm that this pull request has done the following:

  • Tests added
  • Documentation added (where applicable)
  • Description in a {pr}.thing.md file in the directory changelog added - see changelog/README.md for details

Description

Updated to a more recent ruff version in linting and fixed all of its complaints.

Potentially breaking change: Some ValueErrors are now TypeErrors

@codecov

codecov Bot commented Sep 1, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.16981% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.30%. Comparing base (73c5ca6) to head (971c23b).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
primap2/_data_format.py 75.00% 1 Missing ⚠️
primap2/csg/_wrapper.py 83.33% 1 Missing ⚠️
primap2/pm2io/_data_reading.py 96.66% 1 Missing ⚠️
Additional details and impacted files
@@                    Coverage Diff                     @@
##           fix/timestamp-in-tests     #383      +/-   ##
==========================================================
+ Coverage                   97.29%   97.30%   +0.01%     
==========================================================
  Files                          54       54              
  Lines                        5361     5350      -11     
==========================================================
- Hits                         5216     5206      -10     
+ Misses                        145      144       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@fuchsi-huber
fuchsi-huber merged commit 09e617e into main Sep 2, 2026
13 of 14 checks passed
@fuchsi-huber
fuchsi-huber deleted the cleanup/ruff branch September 2, 2026 10:44
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