-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
MAINT: Port _timelex in codebase #24520
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
Conversation
7b88b7b
to
2a1500d
Compare
Codecov Report
@@ Coverage Diff @@
## master #24520 +/- ##
=======================================
Coverage 31.88% 31.88%
=======================================
Files 166 166
Lines 52434 52434
=======================================
Hits 16718 16718
Misses 35716 35716
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #24520 +/- ##
=======================================
Coverage 31.88% 31.88%
=======================================
Files 166 166
Lines 52413 52413
=======================================
Hits 16714 16714
Misses 35699 35699
Continue to review full report at Codecov.
|
General approach seems right to me. I'll try to keep you informed if we make major changes in |
2a1500d
to
29febe3
Compare
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.
LGTM. I agree with pretty much all the comments made so far.
Removes the DeprecationWarning raised by dateutil because it's a private class. Implementation taken from the following PR: dateutil/dateutil#732 Closes pandas-devgh-21322.
@jreback : Starting 2019 off on a good note with an all green PR! 🎉 All requested changes have been made, and all is green. PTAL. |
thanks @gfyoung @jbrockmendel and @pganssle |
* upstream/master: BUG: output formatting with to_html(), index=False and/or index_names=False (pandas-dev#22579, pandas-dev#22747) (pandas-dev#22655) MAINT: Port _timelex in codebase (pandas-dev#24520) Implement unique+array parts of 24024 (pandas-dev#24527) Integer NA docs (pandas-dev#23617)
* MAINT: Port _timelex in codebase Removes the DeprecationWarning raised by dateutil because it's a private class. Implementation taken from the following PR: dateutil/dateutil#732 Closes pandas-devgh-21322. * MAINT: Drop _timelex compat for dateutil 2.2
* MAINT: Port _timelex in codebase Removes the DeprecationWarning raised by dateutil because it's a private class. Implementation taken from the following PR: dateutil/dateutil#732 Closes pandas-devgh-21322. * MAINT: Drop _timelex compat for dateutil 2.2
Removes the
DeprecationWarning
raised bydateutil
because it's a private class.Implementation taken from the following PR:
dateutil/dateutil#732
Closes #21322.
cc @pganssle