w3c / activitystreams

Activity Streams 2.0
https://www.w3.org/TR/activitystreams-core/
Other
278 stars 62 forks source link

ABNF as2-partial-time accept time-secfrac without time-second #488

Open artlog opened 6 years ago

artlog commented 6 years ago

Please Indicate One:

Please Describe the Issue:

for ABNF in Date and Time :

as2-partial-time = time-hour ":" time-minute [":" time-second] [time-secfrac]

shouldn't it be

as2-partial-time = time-hour ":" time-minute [":" time-second [time-secfrac] ]

since time-secfrac has few interest if time-second is not provided.

gobengo commented 5 years ago

Good eye. Though conceptually you can add a fraction of a second to a minute without first needing to add a second.

I don't think there are any inconsistencies with the way it is that would warrant a backward-incompatible change or adding to errata.

Again, though, good eye.

evanp commented 10 months ago

This seems like an error to me. I added an erratum here:

https://github.com/w3c/activitystreams/pull/550