-
Notifications
You must be signed in to change notification settings - Fork 12
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
Don't allow negative "nth" specifications in "nth" ranges #68
Comments
I can't think of any real case where this made sense, i.e. I can't think of an opening hours sign that could be written that way...
If negative "nth" actually appear in the wild, I think the chance is higher that it is a typo rather than what the contributor meant. However, a less ludicrous opening hours sign may say
If I understood the specification correctly, writing |
There are two issues: on the one hand there are no month day ranges that are separate from dates. What is supported is only Jan 1-31 which simply is short hand for Jan 1 - Jan 31, so support would require month day ranges as an independent construct. Not sure if that wouldn't create even more ambiguity in parsing (for example with that shorthand). The 2nd minor issue is adding an interval specification, that is unlikely to be problematic on it's own. |
I don't see how |
Anyway, regarding this topic, for reference, the specification also does not allow negative nth specifications in ranges. E.g.
|
Does allowing this make any sense? If it does it should definitely throw an exception in strict mode.
The text was updated successfully, but these errors were encountered: