Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch linting to ruff #266

Merged
merged 1 commit into from
Jan 31, 2024
Merged

Switch linting to ruff #266

merged 1 commit into from
Jan 31, 2024

Conversation

ffl096
Copy link
Member

@ffl096 ffl096 commented Jan 31, 2024

@ffl096 ffl096 self-assigned this Jan 31, 2024
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@ffl096 ffl096 force-pushed the frantzen-ruff branch 2 times, most recently from a040909 to 0b95211 Compare January 31, 2024 08:55
Copy link

codecov bot commented Jan 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (96a0b24) 97.45% compared to head (641cf21) 97.45%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #266   +/-   ##
=======================================
  Coverage   97.45%   97.45%           
=======================================
  Files          58       58           
  Lines        2120     2120           
=======================================
  Hits         2066     2066           
  Misses         54       54           

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

@ffl096 ffl096 requested a review from ninamiolane January 31, 2024 09:10
@ninamiolane ninamiolane merged commit 7f3c2ef into main Jan 31, 2024
24 checks passed
@ffl096 ffl096 deleted the frantzen-ruff branch January 31, 2024 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants