Skip to content

Conversation

@simonjayhawkins
Copy link
Member

@simonjayhawkins simonjayhawkins commented Jan 6, 2019

this refactor further limits responsibilities of functions/module and should allow a simpler implementation for fixes of #17004, #9690 etc.

@codecov
Copy link

codecov bot commented Jan 6, 2019

Codecov Report

Merging #24651 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #24651      +/-   ##
==========================================
+ Coverage   92.37%   92.37%   +<.01%     
==========================================
  Files         166      166              
  Lines       52384    52386       +2     
==========================================
+ Hits        48390    48392       +2     
  Misses       3994     3994
Flag Coverage Δ
#multiple 90.8% <100%> (ø) ⬆️
#single 43.02% <0%> (-0.01%) ⬇️
Impacted Files Coverage Δ
pandas/io/formats/html.py 99.34% <100%> (ø) ⬆️
pandas/io/formats/format.py 97.98% <100%> (ø) ⬆️

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 88ca968...ef62b8f. Read the comment docs.

@codecov
Copy link

codecov bot commented Jan 6, 2019

Codecov Report

Merging #24651 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #24651      +/-   ##
==========================================
+ Coverage   92.37%   92.37%   +<.01%     
==========================================
  Files         166      166              
  Lines       52384    52386       +2     
==========================================
+ Hits        48390    48393       +3     
+ Misses       3994     3993       -1
Flag Coverage Δ
#multiple 90.8% <100%> (ø) ⬆️
#single 43.01% <0%> (-0.01%) ⬇️
Impacted Files Coverage Δ
pandas/io/formats/html.py 99.34% <100%> (ø) ⬆️
pandas/io/formats/format.py 97.98% <100%> (ø) ⬆️
pandas/util/testing.py 88.09% <0%> (+0.09%) ⬆️

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 88ca968...2b0e24e. Read the comment docs.

Copy link
Contributor

@jreback jreback left a comment

Choose a reason for hiding this comment

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

@jreback jreback added Refactor Internal refactoring of code IO HTML read_html, to_html, Styler.apply, Styler.applymap labels Jan 6, 2019
@jreback jreback added this to the 0.24.0 milestone Jan 8, 2019
@jreback jreback merged commit f57bd72 into pandas-dev:master Jan 8, 2019
@jreback
Copy link
Contributor

jreback commented Jan 8, 2019

thanks @simonjayhawkins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IO HTML read_html, to_html, Styler.apply, Styler.applymap Refactor Internal refactoring of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants