Skip to content

Conversation

@gwrome
Copy link
Contributor

@gwrome gwrome commented May 6, 2019

"""

style_backup = dict()
style_backup = dict() # type: Dict
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is this variable still needed? I searched the codebase for style_backup, and it doesn't appear anywhere else. It might be some vestigial code.

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah looks like you can remove it

@codecov
Copy link

codecov bot commented May 7, 2019

Codecov Report

Merging #26300 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #26300      +/-   ##
==========================================
- Coverage   91.98%   91.98%   -0.01%     
==========================================
  Files         175      175              
  Lines       52374    52373       -1     
==========================================
- Hits        48178    48173       -5     
- Misses       4196     4200       +4
Flag Coverage Δ
#multiple 90.53% <ø> (-0.01%) ⬇️
#single 40.72% <ø> (-0.15%) ⬇️
Impacted Files Coverage Δ
pandas/core/config_init.py 96.94% <ø> (-0.03%) ⬇️
pandas/io/gbq.py 78.94% <0%> (-10.53%) ⬇️
pandas/core/frame.py 96.9% <0%> (-0.12%) ⬇️
pandas/core/base.py 97.98% <0%> (ø) ⬆️

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 94c8c94...7b363f6. Read the comment docs.

@jreback jreback added the Typing type annotations, mypy/pyright type checking label May 7, 2019
@jreback jreback added this to the 0.25.0 milestone May 7, 2019
@jreback
Copy link
Contributor

jreback commented May 7, 2019

lgtm. ping on green.

@gwrome
Copy link
Contributor Author

gwrome commented May 7, 2019

@jreback The board is green

@jreback jreback merged commit 07cea19 into pandas-dev:master May 7, 2019
@jreback
Copy link
Contributor

jreback commented May 7, 2019

thanks @gwrome

@gwrome gwrome deleted the blacklist-config_init branch May 9, 2019 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Typing type annotations, mypy/pyright type checking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants