mwouts / itables

Pandas DataFrames as Interactive DataTables
https://mwouts.github.io/itables/
MIT License
730 stars 54 forks source link

Further simplify _format_column #119

Closed mwouts closed 1 year ago

mwouts commented 1 year ago

and introduce warn_on_unexpected_types

Follow-up on #112 #117

codecov-commenter commented 1 year ago

Codecov Report

Merging #119 (4760778) into main (c32000c) will increase coverage by 0.26%. The diff coverage is 88.67%.

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   92.73%   92.99%   +0.26%     
==========================================
  Files          19       21       +2     
  Lines         509      557      +48     
==========================================
+ Hits          472      518      +46     
- Misses         37       39       +2     
Impacted Files Coverage Δ
tests/test_update_samples.py 82.35% <66.66%> (ø)
itables/javascript.py 84.07% <68.75%> (-0.17%) :arrow_down:
itables/datatables_format.py 84.21% <84.21%> (ø)
itables/downsample.py 98.03% <100.00%> (ø)
itables/options.py 100.00% <100.00%> (ø)
itables/sample_dfs.py 100.00% <100.00%> (ø)
itables/utils.py 100.00% <100.00%> (ø)
itables/version.py 100.00% <100.00%> (ø)
tests/test_changelog.py 100.00% <100.00%> (ø)
tests/test_connected_notebook_is_small.py 100.00% <100.00%> (ø)
... and 2 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

lgtm-com[bot] commented 1 year ago

This pull request fixes 1 alert when merging 514a167a7be09319db4481396deead9582b98a04 into 5a37ca66746e1b460f6d42252e7b7153dec45de2 - view on LGTM.com

fixed alerts:

lgtm-com[bot] commented 1 year ago

This pull request fixes 1 alert when merging 4760778307cf9921a760193a10a228d07f47768e into 5a37ca66746e1b460f6d42252e7b7153dec45de2 - view on LGTM.com

fixed alerts: