Repository navigation
Added suppress warning option - #582
EliusSolis wants to merge 4 commits into
Conversation
Linting results by Pylint:Your code has been rated at 8.31/10 (previous run: 8.34/10, -0.03) |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## RC_v1.6.x #582 +/- ##
==========================================
Coverage 66.05% 66.05%
==========================================
Files 28 28
Lines 4251 4251
==========================================
Hits 2808 2808
Misses 1443 1443
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
w-k-jones
left a comment
There was a problem hiding this comment.
Hi @EliusSolis , thanks for your first contribution! Could you format the code using the black library, and add a test to make sure that the warning is issued when suppress_warnings=False and no warning when suppress_warnings=True. Thanks!
w-k-jones
left a comment
There was a problem hiding this comment.
Hi @EliusSolis thanks for these changes, I'm happy to merge this now. Thanks again for contributing!
Resolves #558