Skip to content

gh-124549: Warn on conflicting directives in datetime.*.strptime - #132524

Open
StanFromIreland wants to merge 4 commits into
python:mainfrom
StanFromIreland:issue-124549
Open

gh-124549: Warn on conflicting directives in datetime.*.strptime#132524
StanFromIreland wants to merge 4 commits into
python:mainfrom
StanFromIreland:issue-124549

Conversation

@StanFromIreland

@StanFromIrelandStanFromIreland commented Apr 14, 2025•
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

I agree with @Mariatta:

I created a PR to document this weird behavior. I think it's still worth raising a warning though. #125009

And so I extended your PR, I hope you don't mind :-)

📚 Documentation preview 📚: https://cpython-previews--132524.org.readthedocs.build/

@StanFromIreland@Mariatta
Raise a warning when passing conflicting directives
eb63f10
Co-authored-by: Mariatta <Mariatta@users.noreply.github.com>
@bedevere-appbedevere-appBot mentioned this pull request Apr 14, 2025
Open
@StanFromIrelandStanFromIreland changed the title gh-124549: Warn on conflicting directivesgh-124549: Warn on conflicting directives in datetime.*.strptimeApr 14, 2025
jhohm
jhohm suggested changes May 19, 2025
View reviewed changes
Comment threadDoc/library/datetime.rst
Comment threadLib/_strptime.py
Mariatta
Mariatta reviewed Feb 3, 2026
View reviewed changes
Comment threadLib/_strptime.py
@Mariatta

Mariatta commented Feb 3, 2026

Copy link
Copy Markdown
Member

Please resolve the conflict. Thanks.

@github-actions

github-actionsBot commented Apr 23, 2026

Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actionsgithub-actionsBot added the stale Stale PR or inactive for long period of time. label Apr 23, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pgansslepganssleAwaiting requested review from pgansslepganssle is a code owner
@abalkinabalkinAwaiting requested review from abalkin
@MariattaMariattaAwaiting requested review from Mariatta
1 more reviewer
@jhohmjhohmjhohm requested changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

awaiting core reviewstaleStale PR or inactive for long period of time.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@StanFromIreland@Mariatta@jhohm