-
Notifications
You must be signed in to change notification settings - Fork 442
code cleanup and reformat #423
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
Conversation
This pull request introduces 3 alerts and fixes 3 when merging 037ee3c into 0f1ff4c - view on LGTM.com new alerts:
fixed alerts:
|
This pull request fixes 3 alerts when merging e70f362 into 0f1ff4c - view on LGTM.com fixed alerts:
|
This pull request fixes 3 alerts when merging 30f0814 into 0f1ff4c - view on LGTM.com fixed alerts:
|
Codecov Report
@@ Coverage Diff @@
## master #423 +/- ##
==========================================
- Coverage 86.75% 86.59% -0.17%
==========================================
Files 12 12
Lines 1533 1537 +4
==========================================
+ Hits 1330 1331 +1
- Misses 203 206 +3
Continue to review full report at Codecov.
|
This pull request fixes 3 alerts when merging ae4757d into 0f1ff4c - view on LGTM.com fixed alerts:
|
for the |
Co-Authored-By: Inada Naoki <[email protected]>
This pull request fixes 3 alerts when merging eab8bcb into 0f1ff4c - view on LGTM.com fixed alerts:
|
Thank you. |
in this PR.
I did
#noqa: E501 to it
flake8
should not complain about it. just keep this here, since black recommendu"unicode_string"
->"unicode"
, since python2 support has been droppeddocs/conf.py
alone, since there are some#comment_me = True
codes for the configurationblack
andflake8
totravis