-
Notifications
You must be signed in to change notification settings - Fork 57
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
refactor utils with pylint #683
Conversation
Codecov Report
@@ Coverage Diff @@
## main #683 +/- ##
=======================================
Coverage 89.93% 89.93%
=======================================
Files 192 192
Lines 7315 7315
Branches 1157 1157
=======================================
Hits 6579 6579
Misses 491 491
Partials 245 245
|
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.
Thanks!
failing black CICD pipeline because of |
Possibly related issue: psf/black#2634 |
Hex @akapoor13, it seems that due to the other pylint-PRs being merged more quickly than this one, merge conflicts were created. Please ping me once they are resolved and I will make sure to merge the PR swiftly :) |
Hey @pnkraemer think we're good to go now |
In a Nutshell
Fix pylint messages in
probnum.utils
Detailed Description
Refactors
probnum.utils
to comply with pylintline-too-long
,no-else-return
andelse-if-used
.Related Issues
Closes #666