-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: read_sql date parsing #24500
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
CLN: read_sql date parsing #24500
Conversation
Codecov Report
@@ Coverage Diff @@
## master #24500 +/- ##
=======================================
Coverage 92.3% 92.3%
=======================================
Files 166 166
Lines 52412 52412
=======================================
Hits 48381 48381
Misses 4031 4031
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #24500 +/- ##
===========================================
- Coverage 92.3% 31.88% -60.42%
===========================================
Files 166 166
Lines 52412 52421 +9
===========================================
- Hits 48381 16717 -31664
- Misses 4031 35704 +31673
Continue to review full report at Codecov.
|
looks fine. is there any user api change here? (meaning do we need a whatsnew note) |
No API change at all, just reorganization. The linked issue already had an existing test, but the test wasn't as strict as it should have been. |
thanks! |
git diff upstream/master -u -- "*.py" | flake8 --diff
Cleaning up the date parsing portion of the read_sql functionality. Also checking that #15119 can be closed as we are already testing the solution.