Skip to content

gh-127552: Remove comment questioning 4-digit restriction for ‘Y’ in datetime.strptime patterns #127590

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

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

bombs-kim
Copy link
Contributor

@bombs-kim bombs-kim commented Dec 4, 2024

This PR removes the comment questioning 4-digit restriction for ‘Y’ in datetime.strptime patterns. The comment is more than 20 years old and practically obsolete at this point.

@brettcannon brettcannon added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes and removed needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Dec 4, 2024
@brettcannon brettcannon merged commit 51cfa56 into python:main Dec 4, 2024
52 checks passed
@brettcannon
Copy link
Member

Thanks!

@picnixz
Copy link
Member

picnixz commented Dec 5, 2024

@brettcannon any reason not to backport this (or was it a misclick)?

@brettcannon
Copy link
Member

@brettcannon any reason not to backport this (or was it a misclick)?

I decided it simply wasn't worth it in the end (more churn than necessary). Feel free to if you want to.

@picnixz
Copy link
Member

picnixz commented Dec 5, 2024

I'll just see if we can have an automatic backport. If not, I'll do the backports manually! (tomorrow)

@picnixz picnixz added needs backport to 3.12 only security fixes needs backport to 3.13 bugs and security fixes labels Dec 5, 2024
@miss-islington-app
Copy link

Thanks @bombs-kim for the PR, and @brettcannon for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Thanks @bombs-kim for the PR, and @brettcannon for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 5, 2024
…Y’ in datetime.strptime patterns (pythonGH-127590)

The code has required 4 digits for the year since its inclusion in the stdlib in 2002 (over 22 years ago as of this commit).
(cherry picked from commit 51cfa56)

Co-authored-by: Beomsoo Kim <beoms424@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 5, 2024
…Y’ in datetime.strptime patterns (pythonGH-127590)

The code has required 4 digits for the year since its inclusion in the stdlib in 2002 (over 22 years ago as of this commit).
(cherry picked from commit 51cfa56)

Co-authored-by: Beomsoo Kim <beoms424@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Dec 5, 2024

GH-127649 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Dec 5, 2024
@bedevere-app
Copy link

bedevere-app bot commented Dec 5, 2024

GH-127650 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Dec 5, 2024
brettcannon pushed a commit that referenced this pull request Dec 6, 2024
…‘Y’ in datetime.strptime patterns (GH-127590) GH-127650)

gh-127552: Remove comment questioning 4-digit restriction for ‘Y’ in datetime.strptime patterns (GH-127590)

The code has required 4 digits for the year since its inclusion in the stdlib in 2002 (over 22 years ago as of this commit).
(cherry picked from commit 51cfa56)

Co-authored-by: Beomsoo Kim <beoms424@gmail.com>
cfbolz pushed a commit that referenced this pull request Dec 10, 2024
…‘Y’ in datetime.strptime patterns (GH-127590) (#127649)

gh-127552: Remove comment questioning 4-digit restriction for ‘Y’ in datetime.strptime patterns (GH-127590)

The code has required 4 digits for the year since its inclusion in the stdlib in 2002 (over 22 years ago as of this commit).
(cherry picked from commit 51cfa56)

Co-authored-by: Beomsoo Kim <beoms424@gmail.com>
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
srinivasreddy pushed a commit to srinivasreddy/cpython that referenced this pull request Jan 8, 2025
…Y’ in datetime.strptime patterns (python#127590)

The code has required 4 digits for the year since its inclusion in the stdlib in 2002 (over 22 years ago as of this commit).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants