Skip to content
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

Fix import format at pandas/tests/io/plotting directory #24580

Merged
merged 3 commits into from
Jan 4, 2019

Conversation

mwoss
Copy link
Contributor

@mwoss mwoss commented Jan 3, 2019

Ran isort --recursive pandas/tests/io/plotting and then checked imports using isort --recursive --check-only pandas/tests/io/plotting

@jreback jreback added the Code Style Code style, linting, code_checks label Jan 3, 2019

converter = pytest.importorskip('pandas.plotting._converter')
from pandas.plotting import (register_matplotlib_converters,
deregister_matplotlib_converters)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I imagine you will need to use an isort:skip here

@codecov
Copy link

codecov bot commented Jan 4, 2019

Codecov Report

Merging #24580 into master will increase coverage by 49.31%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #24580       +/-   ##
===========================================
+ Coverage   43.05%   92.36%   +49.31%     
===========================================
  Files         166      166               
  Lines       52490    52494        +4     
===========================================
+ Hits        22598    48485    +25887     
+ Misses      29892     4009    -25883
Flag Coverage Δ
#multiple 90.79% <ø> (?)
#single 43.04% <ø> (-0.02%) ⬇️
Impacted Files Coverage Δ
pandas/core/computation/pytables.py 92.37% <0%> (+0.3%) ⬆️
pandas/io/pytables.py 92.31% <0%> (+0.92%) ⬆️
pandas/util/_test_decorators.py 90.54% <0%> (+2.7%) ⬆️
pandas/compat/__init__.py 57.91% <0%> (+8.1%) ⬆️
pandas/core/config_init.py 99.24% <0%> (+9.84%) ⬆️
pandas/core/reshape/util.py 100% <0%> (+11.53%) ⬆️
pandas/core/api.py 100% <0%> (+13.33%) ⬆️
pandas/compat/numpy/__init__.py 92.85% <0%> (+14.28%) ⬆️
pandas/core/computation/common.py 85.71% <0%> (+14.28%) ⬆️
pandas/core/indexes/api.py 99% <0%> (+14.85%) ⬆️
... and 124 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9175387...6a740c8. Read the comment docs.

@codecov
Copy link

codecov bot commented Jan 4, 2019

Codecov Report

Merging #24580 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #24580   +/-   ##
=======================================
  Coverage   92.38%   92.38%           
=======================================
  Files         166      166           
  Lines       52490    52490           
=======================================
  Hits        48493    48493           
  Misses       3997     3997
Flag Coverage Δ
#multiple 90.81% <ø> (ø) ⬆️
#single 43.05% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9175387...6a740c8. Read the comment docs.

@jreback jreback added this to the 0.24.0 milestone Jan 4, 2019
@mwoss
Copy link
Contributor Author

mwoss commented Jan 4, 2019

@jreback passed

@jreback jreback merged commit a8f97c1 into pandas-dev:master Jan 4, 2019
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Pingviinituutti pushed a commit to Pingviinituutti/pandas that referenced this pull request Feb 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Style Code style, linting, code_checks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants