Skip to content

Update to ruff 0.16.x, make required fixes - #132

Merged
chrisjonesBSU merged 1 commit into
cmelab:mainfrom
chrisjonesBSU:main
Sep 14, 2026
Merged

chrisjonesBSU merged 1 commit into
cmelab:mainfrom
chrisjonesBSU:main

Conversation

@chrisjonesBSU

Copy link
Copy Markdown
Member

ruff (used by precommit) recently updated from 0.15.x to 0.16.x, which cause quite a bit of errors. This PR cleans those up.

@chrisjonesBSU chrisjonesBSU added the maintenance General CI/maintenance changes label Sep 14, 2026
@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.57143% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.01%. Comparing base (4a1bb72) to head (7315d22).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
grits/finegrain.py 60.00% 2 Missing ⚠️
grits/utils.py 75.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #132      +/-   ##
==========================================
+ Coverage   94.98%   95.01%   +0.02%     
==========================================
  Files           4        4              
  Lines         459      461       +2     
==========================================
+ Hits          436      438       +2     
  Misses         23       23              
Files with missing lines Coverage Δ
grits/__init__.py 77.77% <ø> (ø)
grits/coarsegrain.py 95.94% <100.00%> (ø)
grits/utils.py 97.52% <75.00%> (+0.04%) ⬆️
grits/finegrain.py 88.33% <60.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@chrisjonesBSU
chrisjonesBSU merged commit ff1f77e into cmelab:main Sep 14, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance General CI/maintenance changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant