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
Removing path and line number from warnings. #447
Removing path and line number from warnings. #447
Conversation
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
Merged build finished. Test PASSed. |
Test PASSed. |
* Declutters the warning message so the message is easier to see * Removes the `warnings.warn(message)` from the message as well. * Difference between old and new shown below: Previous: ``` /Users/DevinPetersohn/software_builds/modin/modin/error_message.py:32: UserWarning: `cov` defaulting to pandas implementation. To request implementation, send an email to feature_requests@modin.org. warnings.warn(message) ``` This commit: ``` WARN: `cov` defaulting to pandas implementation. To request implementation, send an email to feature_requests@modin.org. ```
d6dbe91
to
e4f073c
Compare
Merged build finished. Test PASSed. |
Test PASSed. |
Codecov Report
@@ Coverage Diff @@
## master #447 +/- ##
==========================================
+ Coverage 83.23% 83.24% +0.01%
==========================================
Files 33 33
Lines 4498 4501 +3
==========================================
+ Hits 3744 3747 +3
Misses 754 754
Continue to review full report at Codecov.
|
warnings.warn(message)
from the message as well.Previous:
This commit:
git diff upstream/master -u -- "*.py" | flake8 --diff
black --check modin/