-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add cron character explanation and give examples of different schedules #17
Conversation
Signed-off-by: tpmccallum [email protected] Signed-off-by: Timothy McCallum <[email protected]>
Signed-off-by: tpmccallum [email protected] Signed-off-by: Timothy McCallum <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some nits but otherwise LGTM
README.md
Outdated
The `cron_expression` format for scheduling is as follows: | ||
|
||
```text | ||
sec min hour day of month month day of week year |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This isn't actually part of the expression is it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, yes. I have updated so that we refer to the sec
, min
etc. as fields. I also use a bit of ascii-art inspired text to show each of the fields.
Signed-off-by: tpmccallum [email protected] Co-authored-by: itowlson <[email protected]> Signed-off-by: Timothy McCallum <[email protected]>
Signed-off-by: tpmccallum [email protected] Co-authored-by: itowlson <[email protected]> Signed-off-by: Timothy McCallum <[email protected]>
Signed-off-by: tpmccallum [email protected] Co-authored-by: itowlson <[email protected]> Signed-off-by: Timothy McCallum <[email protected]>
Signed-off-by: tpmccallum [email protected] Signed-off-by: Timothy McCallum <[email protected]>
Signed-off-by: tpmccallum [email protected]