-
Notifications
You must be signed in to change notification settings - Fork 6
vcp: Clarify ordinal dates in parsed result #27
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
Comments
I think it should indeed be preserved. Where you discover the specific date value earlier it explicitly says to “then use that as the date value”:
So if you found |
Current status of parsers Go: accepts ordinal dates and converts to MM-DD in the final output. |
The list of possible VCP assembled output syntaxes was intended to be comprehensive. Thus ordinal ISO dates should be normalized to YYYY-MM-DD. The spec should be explicit about this since it appears parsers disagree. Assembled dates must be normalized YYYY-MM-DD, because parsed output consuming code likely expects that, either way, simpler for JSON consuming code. (Originally published at: http://tantek.com/2018/084/t4/) |
+1 for normalised to YYYY-MM-DD |
Proposed text: Change this line:
To:
|
The specification lists ordinal date format
YYYY-DDD
as a validvalue
.My reading of this^ section is the ordinal date format should be preserved in the assembled datetime. I wanted to confirm that's correct, though, since the only results listed are
YYYY-MM-DD
variants:If it should be preserved, I'd recommend we add some examples to that list.
The text was updated successfully, but these errors were encountered: