-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
centralize and de-privatize month/day name constants #18728
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
I've got some debugging to do here and in two other places. In the interim, there's a bugfix in #18376 that I'm really eager to get closed. |
Codecov Report
@@ Coverage Diff @@
## master #18728 +/- ##
==========================================
- Coverage 91.61% 91.59% -0.02%
==========================================
Files 153 153
Lines 51339 51342 +3
==========================================
- Hits 47034 47028 -6
- Misses 4305 4314 +9
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #18728 +/- ##
==========================================
- Coverage 91.61% 91.59% -0.02%
==========================================
Files 153 153
Lines 51361 51364 +3
==========================================
- Hits 47053 47047 -6
- Misses 4308 4317 +9
Continue to review full report at Codecov.
|
circleCI is a timeout, appveyor looks unrelated, goes through fine locally:
|
@jbrockmendel pls add @mroeschke suggestions to the list, whomever wants can do a PR to centralize those as well! |
git diff upstream/master -u -- "*.py" | flake8 --diff