-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
14350 #18380
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
14350 #18380
Conversation
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
thanks! |
@mpenkov in general, can you try to give PRs a more informative title ? |
@@ -26,6 +26,7 @@ Other Enhancements | |||
- :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 |
There was a problem hiding this comment.
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
?
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
git diff upstream/master -u -- "*.py" | flake8 --diff