Skip to content

Conversation

mpenkov
Copy link
Contributor

@mpenkov mpenkov commented Nov 20, 2017

@codecov
Copy link

codecov bot commented Nov 20, 2017

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #18380       +/-   ##
===========================================
+ Coverage   39.61%   91.34%   +51.72%     
===========================================
  Files         164      164               
  Lines       49799    49799               
===========================================
+ Hits        19728    45487    +25759     
+ Misses      30071     4312    -25759
Flag Coverage Δ
#multiple 89.14% <ø> (?)
#single 39.55% <ø> (-0.07%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/computation/pytables.py 92.37% <0%> (+0.3%) ⬆️
pandas/io/pytables.py 92.84% <0%> (+0.97%) ⬆️
pandas/core/config_init.py 98.34% <0%> (+4.95%) ⬆️
pandas/io/formats/console.py 69.69% <0%> (+6.06%) ⬆️
pandas/compat/__init__.py 58.1% <0%> (+8.1%) ⬆️
pandas/core/reshape/util.py 90.32% <0%> (+12.9%) ⬆️
pandas/core/computation/common.py 85.71% <0%> (+14.28%) ⬆️
pandas/compat/numpy/__init__.py 93.93% <0%> (+18.18%) ⬆️
pandas/core/indexes/api.py 98.78% <0%> (+19.51%) ⬆️
... and 113 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 1915ffc...9777771. Read the comment docs.

@jreback jreback added the Error Reporting Incorrect or improved errors from pandas label Nov 20, 2017
@jreback jreback added this to the 0.22.0 milestone Nov 20, 2017
@jreback jreback merged commit f85cdfd into pandas-dev:master Nov 20, 2017
@jreback
Copy link
Contributor

jreback commented Nov 20, 2017

thanks!

@mpenkov mpenkov deleted the 14350 branch November 20, 2017 11:11
@jorisvandenbossche
Copy link
Member

@mpenkov in general, can you try to give PRs a more informative title ?

- :func:`pandas.tseries.frequencies.to_offset` now accepts leading '+' signs e.g. '+1h'. (:issue:`18171`)
- :class:`pandas.io.formats.style.Styler` now has method ``hide_index()`` to determine whether the index will be rendered in ouptut (:issue:`14194`)
- :class:`pandas.io.formats.style.Styler` now has method ``hide_columns()`` to determine whether columns will be hidden in output (:issue:`14194`)
- Improved wording of ValueError raised in :func:`Timestamp.tz_localize` function
Copy link
Member

Choose a reason for hiding this comment

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

Why is this mentioning tz_localize? Wasn't this about to_datetime ?

Copy link
Contributor Author

@mpenkov mpenkov Nov 20, 2017

Choose a reason for hiding this comment

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

You're right, I specified the wrong function name in whatsnew. I pushed an update just now. Do I need to create a new PR for it?

Copy link
Contributor

Choose a reason for hiding this comment

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

hah i merged already
i’ll fix

jreback added a commit that referenced this pull request Nov 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Error Reporting Incorrect or improved errors from pandas

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DOC/ERR: update error message / doc-string for to_datetime with non-convertible object and unit kw

3 participants