Skip to content

Conversation

@jyalim
Copy link
Contributor

@jyalim jyalim commented Oct 27, 2025

Incrementally builds C99+ datetime support with addition of %F with single line of code. Also adds two tests: one for %F and another for previously supported %T.

Passed all tests (especially changes to new one).

Passed patchcheck.

@bedevere-app
Copy link

bedevere-app bot commented Oct 27, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@python-cla-bot
Copy link

python-cla-bot bot commented Oct 27, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

@StanFromIreland StanFromIreland changed the title gh-140644: provide support for %F token in strptime; add test for it and %T gh-140715: provide support for %F token in strptime; add test for it and %T Oct 28, 2025
@StanFromIreland
Copy link
Member

I moved this under #140715, can you please update the blurb filename.

@jyalim
Copy link
Contributor Author

jyalim commented Oct 28, 2025

I moved this under #140715, can you please update the blurb filename.

I think this is done!

Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

Please add them to this tuple:

for directive in ('a', 'A', 'b', 'B', 'c', 'd', 'H', 'I',
'j', 'm', 'M', 'p', 'S',
'U', 'w', 'W', 'x', 'X', 'y', 'Y', 'Z', '%'):

Also, please update the docs removing the (0) note from its row.

The implementation looks good, just a few small things and this will be ready :-)

@jyalim
Copy link
Contributor Author

jyalim commented Feb 7, 2026

Thank you, @StanFromIreland, all changes have been made as requested!

@jyalim
Copy link
Contributor Author

jyalim commented Feb 8, 2026

Thank you again, @StanFromIreland, changes have been made.

Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

Last few things.

jyalim and others added 7 commits February 8, 2026 13:30
style alignment, add space after function arguments as requested

Co-authored-by: Stan Ulbrych <[email protected]>
style alignment, add space after function arguments as requested

Co-authored-by: Stan Ulbrych <[email protected]>
fix punctuation in docs, as requested

Co-authored-by: Stan Ulbrych <[email protected]>
fix punctuation in docs, as requested

Co-authored-by: Stan Ulbrych <[email protected]>
style alignment, space after functional argument, as requested

Co-authored-by: Stan Ulbrych <[email protected]>
style alignment, space after functional argument, as requested

Co-authored-by: Stan Ulbrych <[email protected]>
@jyalim
Copy link
Contributor Author

jyalim commented Feb 8, 2026

Thanks, @StanFromIreland, the requested changes were implemented!

Copy link
Member

@StanFromIreland StanFromIreland left a comment

Choose a reason for hiding this comment

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

LGTM

@encukou encukou merged commit d99f3fc into python:main Feb 9, 2026
51 checks passed
@StanFromIreland
Copy link
Member

Thanks @jyalim, and congratulations for your first PR! Do you want to open a PR for one of the other format codes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants