Skip to content

Fix outdated comment in _strptime.py #17929

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
Jan 12, 2020

Conversation

cool-RR
Copy link
Contributor

@cool-RR cool-RR commented Jan 9, 2020

Can I please get the tags for skipping bpo and skipping a news item?

Automerge-Triggered-By: @pganssle

@cool-RR
Copy link
Contributor Author

cool-RR commented Jan 9, 2020

@taleinat Could you help with the "skip news" and "skip issue" tags?

@cool-RR cool-RR marked this pull request as ready for review January 9, 2020 21:34
Copy link
Contributor

@taleinat taleinat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@taleinat
Copy link
Contributor

@cool-RR, please check whether this needs back-porting to 3.8 and 3.7.

@cool-RR
Copy link
Contributor Author

cool-RR commented Jan 10, 2020

@taleinat LOL, that code is 17 years old. I'm not sure I understand all the criteria for backporting, but as far as I understand, backporting this fix would be valid, though not very important.

Copy link
Member

@pganssle pganssle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am indifferent about backporting this. I suppose it depends on what we normally do for inaccurate code comments?

@miss-islington miss-islington merged commit 14dbe4b into python:master Jan 12, 2020
@cool-RR cool-RR deleted the 2020-01-09-comment branch January 12, 2020 21:17
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
Can I please get the tags for skipping bpo and skipping a news item?
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.

6 participants