-
Notifications
You must be signed in to change notification settings - Fork 156
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
Lack of minute should not pass the validation #244
Comments
I've stumbled on this too. Similarly, expressions such as just |
Currently, validation of the input expression is rather lax, and if there isn't value specified for the field, it fills it with wildcard. Throwing an error probably would make more sense indeed. Not sure if somebody out there is relying on this "undocumented feature", but changing it to throw an error should be rather trivial. |
Maybe it's a good idea to be part of |
…efaults to false
as the title, when i use expression like
20 15 * *
, it pass the validation and becomes* 20 15 * *
by default, i think the expression without minute should not pass the validation because minute is not optional but required.also, the expression
20 15 * *
does not pass the validation in other package or site like https://crontab.guru/here is the sample, the function
parseExpression
should throw a error but not:The text was updated successfully, but these errors were encountered: