Skip to content

WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime #5578

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.

Sign up for GitHub

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

Closed
matrixise wants to merge 16 commits into from
Closed

WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime #5578

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
ad5ffab
bpo-1100942: Add datetime.time.strptime and datetime.date.strptime
matrixise Feb 7, 2018
50606eb
Follow the PEP7 for the C Code
matrixise May 15, 2018
ab5030f
Remove unused formats
matrixise May 15, 2018
07fb326
Use subtest for the tests
matrixise Oct 17, 2018
010ac8b
Fix the signatures
matrixise Oct 25, 2018
5b58fc1
Update with the recommendation of @pganssle
matrixise Oct 25, 2018
8353a65
Update
matrixise Nov 4, 2018
cff74a9
Add use case where we generate an exception
matrixise Nov 11, 2018
f92a799
Add the Py_UNUSED(ignore) for datetime_gettime & datetime_getdate
matrixise Nov 11, 2018
4c4a7cc
Add a link to the strtime() and strptime() behaviour
matrixise Mar 6, 2019
b006172
Update the blurb entry and add an entry in the Doc/whatsnew/3.8.rst
matrixise Mar 6, 2019
ff7487f
Fix doc
matrixise Mar 6, 2019
a048f0e
Convert the _check_invalid_datetime_spec to the C-API
matrixise Mar 18, 2019
9ac0af4
Support tuple and list for the specs
matrixise Mar 18, 2019
a952bb7
_datetimemodule._check_invalid_datetime_specs only accepts tuple and not
matrixise Mar 24, 2019
412aea2
Fix the code with the recommendation of Martin Panter
matrixise Mar 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add a link to the strtime() and strptime() behaviour
  • Loading branch information
@matrixise
matrixise committed Mar 22, 2019
commit 4c4a7cc09f541d66cf040b948acbb8a366c2e123
8 changes: 5 additions & 3 deletions Doc/library/datetime.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -448,7 +448,8 @@ Other constructors, all class methods:
Return a :class:`date` corresponding to *date_string*, parsed according to
*format*. :exc:`ValueError` is raised if the date string and format can't be
parsed by :meth:`time.strptime`, or if time components are present in the
format string.
format string. For a complete list of formatting directives, see
Copy link
Member

@vadmium vadmium Mar 23, 2019

Choose a reason for hiding this comment

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

Perhaps clarify “time.strptime” does not refer to the strptime function in the time module. The reader may not see the meth RST code, HTML links, etc.

:ref:`strftime-strptime-behavior`.

Copy link
Member

@vstinner vstinner Mar 4, 2019

Choose a reason for hiding this comment

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

Can you please add something like "For a complete list of formatting directives, see strftime() and strptime() Behavior." with a link? (copy the sentence from datetime.datetime.strptime) Just from this doc, I have no idea of what is the expected format.

Copy link
Member Author

@matrixise matrixise Mar 24, 2019

Choose a reason for hiding this comment

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

done

.. versionadded:: 3.8

Expand Down Expand Up @@ -1441,8 +1442,9 @@ Other constructors, all class methods:

Return a :class:`time` corresponding to *date_string, parsed according to
Copy link
Member

@vadmium vadmium Mar 23, 2019

Choose a reason for hiding this comment

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

Why `time` here, but `.time` below (with a dot)?

Copy link
Member Author

@matrixise matrixise Mar 24, 2019

Choose a reason for hiding this comment

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

👍 and there was an other erorr with *date_string*

*format*. :exc:`ValueError` is raised if the date string and format can't be
parsed by :meth:`time.strptime`, if it returns a value which isn't a time tuple,
or if the date part is nonzero.
parsed by :meth:`time.strptime`, if it returns a value which isn't a time
tuple, or if the date part is nonzero. For a complete list of formatting
Copy link
Member

@vadmium vadmium Mar 23, 2019

Choose a reason for hiding this comment

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

“By time.strptime” is redundant, unless you mean to refer to the strptime function in the time module.

What does “the date part is nonzero” mean? I would expect the time class to work without specifying a date, zero or otherwise.

directives, see :ref:`strftime-strptime-behavior`.

.. versionadded:: 3.8

Expand Down