Skip to content
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

Fix skipped next execution time #160

Merged
merged 1 commit into from
Aug 10, 2023
Merged

Conversation

imyip
Copy link

@imyip imyip commented Jul 28, 2023

Fix skipped next execution time
Minute parameters need to be sorted in order

@dragonmantank dragonmantank merged commit ab597c8 into dragonmantank:master Aug 10, 2023
@dragonmantank
Copy link
Owner

Merged, will release shorty. Thanks!

dragonmantank pushed a commit that referenced this pull request Aug 10, 2023
lin-lu added a commit to lin-lu/cron-expression that referenced this pull request Dec 1, 2023
Similar to this PR: dragonmantank#160
The next execution hour is skipped when the hours in the cron expression are not sorted. E.g. the schedule at 10 is skipped if the cron expression is `0 20,10 * * *`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants