-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: replace %s syntax with .format in tslibs/timedeltas #18405
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
FWIW I didn't see what the point of the 'even_day' argument was to _repr_base in timedeltas.pyx. From both the test cases provided and the control flow, unless I'm missing something that argument is no different from passing nothing or None explicitly. Left alone for now but if you want me to remove it won't be a problem |
lgtm. IIRC |
I’ll go ahead and update the docstring to remove. Did you want to keep the test cases for backwards compatibility or would you rather I delete and just stick with the cases for None? |
ok to delete; these are only tests, yes? |
It looks like the tests and one piece of code in pandas/io/formats/format.py at line 2277 to change
To
It looks like there’s already a test case called test_timedelta in the tests/io/formats/test_format.py module to regression test that. Sorry for any formatting issues - sent from my phone. |
c6a4b1c
to
03e4d41
Compare
Codecov Report
@@ Coverage Diff @@
## master #18405 +/- ##
==========================================
- Coverage 91.36% 91.34% -0.02%
==========================================
Files 164 164
Lines 49733 49733
==========================================
- Hits 45439 45430 -9
- Misses 4294 4303 +9
Continue to review full report at Codecov.
|
thanks! |
git diff upstream/master -u -- "*.py" | flake8 --diff