feat: [435] add support for extended cron syntax#877
feat: [435] add support for extended cron syntax#877alexanderkrum wants to merge 3 commits intokelektiv:mainfrom
Conversation
|
@sheerlox need a clarification for this feature:
Do I understand this right? |
|
Not really, because day-of-month and day-of-week work as a union and not an intersection. So basically the Having said that, it would be nice to have the option to enable treating them as intersections if you think that'd be a good way of implementing |
|
But that's a feature of its own, and implementing |
|
@alexanderkrum are you still interested in working on this PR? |
This comment was marked as off-topic.
This comment was marked as off-topic.
Clarifies that when both fields are specified with specific values, they create an OR condition (not AND), following standard POSIX cron behavior. Also mentions the # character syntax being considered in kelektiv#877 as an alternative for patterns like "first Monday of the month". Closes kelektiv#890 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Clarifies that when both fields are specified with specific values, they create an OR condition (not AND), following standard POSIX cron behavior. Also mentions the # character syntax being considered in kelektiv#877 as an alternative for patterns like "first Monday of the month". Closes kelektiv#890 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Description
Related Issue
#435
Motivation and Context
How Has This Been Tested?
Added more unit tests.
Screenshots (if appropriate):
Types of changes
Checklist:
!after the type/scope in the title (see the Conventional Commits standard).