Skip to content

Consistent representation of datetimes and use of calendar attribute#641

Merged
JonathanGregory merged 13 commits into
cf-convention:mainfrom
pvanlaake:main
Jun 11, 2026
Merged

Consistent representation of datetimes and use of calendar attribute#641
JonathanGregory merged 13 commits into
cf-convention:mainfrom
pvanlaake:main

Conversation

@pvanlaake

Copy link
Copy Markdown
Contributor

See issues #624 and #625 for discussion of these changes.

Release checklist

  • history.adoc up to date?
  • Conformance document up to date?

For maintainers

After the merge remember to delete the source branch.
Tags are set at the conclusion of the annual meeting; until then, main always is a draft for the next version.

Comment thread ch04.adoc Outdated
Comment thread ch04.adoc Outdated
Comment thread ch04.adoc
Comment thread ch04.adoc
Comment thread ch04.adoc
pvanlaake and others added 2 commits May 15, 2026 17:25
Co-authored-by: JonathanGregory <j.m.gregory@reading.ac.uk>
Co-authored-by: JonathanGregory <j.m.gregory@reading.ac.uk>
@pvanlaake

Copy link
Copy Markdown
Contributor Author

All comments/suggestions to date have been resolved. Thanks to @JonathanGregory and @larsbarring for their contributions. Inviting others to review and comment.

@JonathanGregory

Copy link
Copy Markdown
Contributor

Thanks, @pvanlaake. I don't think you've changed the time-zone symbol to TZ yet, which you preferred in #625. That seems fine to me: both T and Z individually are problematic, but together they make a solution. :-)

@larsbarring

Copy link
Copy Markdown
Contributor

Yes, TZ is a good solution.

@pvanlaake

Copy link
Copy Markdown
Contributor Author

Thanks for flagging that @JonathanGregory, changes have been made in chapter 4.

@DocOtak

DocOtak commented May 21, 2026

Copy link
Copy Markdown
Member

What ever was done for the __Z__ to __TZ__ change did not actually update the pull request. It will still be __Z__ if merged.

@pvanlaake

Copy link
Copy Markdown
Contributor Author

My naive editing of my own PR... It should now be fixed. That is, if I preview the file I can see TZ consistently, as well as in the diffs

@DocOtak

DocOtak commented May 21, 2026

Copy link
Copy Markdown
Member

Yep, looks fixed to me now!

Deleted RFC 3339 references from lines 251, 277, 283, 284
Removed RFC 3339 reference from bibliography.
Clarify the handling of leading zeros in datetime strings for backwards compatibility and ISO 8601 compliance.
Clarified wording regarding leading zeros in datetime strings for better understanding.
Add requirement for leading zeroes in datetime strings.
@JonathanGregory
JonathanGregory merged commit 0be12fe into cf-convention:main Jun 11, 2026
1 check passed
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.

Use "calendar" attribute in examples Representing datetimes

4 participants