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
DM-40167: Some code cleanups #61
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #61 +/- ##
==========================================
+ Coverage 85.61% 85.86% +0.25%
==========================================
Files 27 27
Lines 3949 3913 -36
Branches 818 801 -17
==========================================
- Hits 3381 3360 -21
+ Misses 441 433 -8
+ Partials 127 120 -7
☔ View full report in Codecov by Sentry. |
This temp directory is never used.
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.
Looks good, a couple of minor comments.
Now uses: with ( a(), b(), ): rather than with a(), b( ):
The changes to |
Only inherit from unittest.TestCase if type checking.
Checklist
doc/changes