-
Notifications
You must be signed in to change notification settings - Fork 32
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
lint code base with ruff
#195
Conversation
Codecov Report
@@ Coverage Diff @@
## main #195 +/- ##
=======================================
- Coverage 88.8% 88.7% -0.1%
=======================================
Files 8 8
Lines 1112 1107 -5
=======================================
- Hits 987 982 -5
Misses 125 125
|
Co-authored-by: Martin Fleischmann <[email protected]>
Co-authored-by: Martin Fleischmann <[email protected]>
Co-authored-by: Martin Fleischmann <[email protected]>
Also, the release action is being triggered for these CI runs. Need to get that under control. |
There was a syntax error in @martinfleis Is this ready to merge following this next passing CI? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Thanks!
This PR resolves #185