-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Widen cron syntax support, update docs #32414
Comments
We could use the Some other libraries which also have the equivalent of above mentioned
|
Take the one which supports all? |
I am confused between these two. |
Cronojs seems "unmaintained", although with no bugs open either. I think it's worth going with a maintained dependency, even if we have to document 1-2 limitations |
Should all the schedule examples in |
Yes good idea |
Describe the proposed change(s).
Discussion: #32369
We should support as much of the cron spec as our parser allows us. I'm not sure why we needed to write our own logic actually, isn't there the ability for us to use a library to tell us "is this now?"
Additionally, our docs on scheduling should be changed to be cron-first and not using
later
syntax by default.The text was updated successfully, but these errors were encountered: