Skip to content

Add 12-hour datetime format parsing - #118

Open
hamzahchaudhry wants to merge 2 commits into
GothenburgBitFactory:masterfrom
hamzahchaudhry:add-12-hour-dateformat
Open

Add 12-hour datetime format parsing#118
hamzahchaudhry wants to merge 2 commits into
GothenburgBitFactory:masterfrom
hamzahchaudhry:add-12-hour-dateformat

Conversation

@hamzahchaudhry

Copy link
Copy Markdown
  • i: minimal 12-hour value, e.g. 1 or 12
  • I: zero-padded 12-hour value, e.g. 01 or 12
  • p: AM/PM marker, case-insensitive when parsing

Ambiguous 12-hour inputs without p are rejected, so 1:30 PM is valid but 1:30 with a 12-hour format is not silently interpreted as AM or PM.

GothenburgBitFactory/taskwarrior#3981

Signed-off-by: hamzah <hamzah.chaudhry@proton.me>
Signed-off-by: hamzah <hamzah.chaudhry@proton.me>

@lauft lauft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

But I think we also need to update the relevant documentation in Taskwarrior and Timewarrior.

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.

2 participants