GH-70647: Remove support for `%d` (and deprecate for `%e`) without year in `strptime()` by StanFromIreland · Pull Request #144570 · python/cpython
Including %e makes sense (an oversight from the original change).
I do not believe pushing this deprecation out to 3.18 is a great idea though... We will have people dealing with production outages on leap day yet again due to bad date parsing around 2028-02-29 if the deprecation has not already shipped in a release that has seen wide adoption before that time.
3.15 will be widely adopted by 2028-02 as it should've shipped in several major long term support os distro releases that people will actually be using by 2028.
3.16 will have some adoption (maybe enough to have forced a fix to the broader ecosystem's errant strptime calls?).
3.17 probably not in wide use.
3.18 won't yet exist.
Most things I like giving a long deprecation grace period on, this one is a ticking time bomb every 4 years that IMNSHO really does need action to be taken in a timely manner. One of the rare cases where forcing the issue is a good thing.
I'm not gonna block pushing it out, but I suggest reconsidering.
Including %e makes sense (an oversight from the original change).
It wasn't implemented at the time.
production outages on leap day
In my opinion, this really depends on what we decide to do to fix the issue. If we are going to always raise an error, that itself will cause disruption, leap day or not, so I'd be in favour of a longer deprecation period.
If we are to change the default year... I haven't considered what side effects that would have. I'd imagine there are people relying on the specific year (
3.15 will be widely adopted by 2028-02 as it should've shipped in several major long term support os distro releases that people will actually be using by 2028.
We can split the removals. Make %d fail now; deprecate %e and plan removal for 3.17.
If we are going to always raise an error, that itself will cause disruption, leap day or not, so I'd be in favour of a longer deprecation period.
The major difference is that if we always raise an error, it will show up in testing environments after a Python upgrade, giving you time to roll back.
If it only fails when the input is a leap day, and you use current date as input, your production app will go down on 2028-02-29.
If we are to change the default year... I haven't considered what side effects that would have. I'd imagine there are people relying on the specific year ( 1900 ).
Yeah, this smells too much like silently returning a wrong value to avoid an exception.
We can split the removals. Make %d fail now; deprecate %e and plan removal for 3.17.
If we are to do it, I'd rather do both at once, %e was illegitimately (i.e. after the feature freeze) added to 3.13/3.14, the first version it technically should be in and is documented in is 3.15.
Sorry, I let this stall.
Mistakes were made. As far as users are concerned, this is in 3.13.
I still think splitting the removal dates is best here. As Greg notes, there's a good reason to remove %d now; %s needs some deprecation period (and I doubt it'll see wide use if it's deprecated now).
Ok, we can do %d in 3.15.
For %s, did you mean %e? I would still lean towards doing both, it was "private" in 3.13/14.
Yeah, %e, sorry.
What do you mean by “private”? It's present in 3.14.0; doc-wise it's covered either by “full set of format codes supported varies across platforms” or PEP 387's “if something is not documented at all, it is not automatically considered private”.
StanFromIreland
changed the title
GH-70647: Include
GH-70647: Remove support for %e in deprecation of strptime() day of month parsing without a year%d (and deprecate for %e) without year in strptime()
Removed support for %d, and deprecated and scheduled removal for %e for 3.17.
I'm not sure how to avoid confusing the docs, should we keep two version changeds (one for deprecation and one for removal) or have just one?
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the docs need further massaging it can be done as followups.
ljfp pushed a commit to ljfp/cpython that referenced this pull request
…out year in `strptime()` (pythonGH-144570) * Add deprecation for %e with no year * schedule `%e` for 3.17, and remove `%d` now