-
Notifications
You must be signed in to change notification settings - Fork 103
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
Add functionality to parse datetimes according C standard format codes. #165
Merged
wimglenn
merged 28 commits into
r1chardj0n3s:master
from
bendichter:parse_flexible_dates
Nov 25, 2023
Merged
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
8237019
Add functionality to parse datetimes according C standard format codes.
bendichter f4b0cbe
use regex to dynamically create specific datetime regex paterns
bendichter c1bee76
change from f-string to str.format syntax
bendichter 28b6336
remove copy import
bendichter 30173ed
Merge branch 'og_master' into parse_flexible_dates
bendichter cd31fc2
allow for colons in time format
bendichter 3d2ea99
allow for more flexible parsing of %z
bendichter 32d83fb
shield Python 2 from timezone features
bendichter acdfeb0
add time parsing
bendichter 5cc7111
uglify code with black
wimglenn f71aa7a
bump version
wimglenn 72591f7
handle and test single digits for day and month
bendichter c3a4f32
Merge remote-tracking branch 'origin/parse_flexible_dates' into parse…
bendichter 5210e18
remove %-j handling
wimglenn fb6d2c0
make j flexible number of digits
bendichter 550f5e5
simplify and reorder, to match docs
wimglenn 8771d3f
just use the map directly
wimglenn fb0c8d9
readability improvements
wimglenn 77328ef
change "tc" to use generic datetime parsing approach
bendichter 11342e1
Merge remote-tracking branch 'origin/parse_flexible_dates' into parse…
bendichter 34edfc8
use new conv variable
bendichter 2c5f905
revert to old logic for tc
bendichter 6af64dd
blacken again
wimglenn b08ed21
blacken again sorry
wimglenn 4b1ba61
new logic:
bendichter fd1a414
Merge remote-tracking branch 'origin/parse_flexible_dates' into parse…
bendichter 44c97bb
doc update
wimglenn 58c998b
test roundtrip every directive
wimglenn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bendichter Somehow we both missed a
raise
here (ref #196)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops. Thanks for catching that