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

Incorrect description for Quartz Cron Expression: * 2,1/31 * * * ? #440

Closed
littleYanzi opened this issue Jul 21, 2020 · 4 comments
Closed
Assignees
Milestone

Comments

@littleYanzi
Copy link

I got the following description from CronDescriptor for Cron Expression: * 2,1/31 * * * ?
at and 2 minutes and every 31 minutes

The description is not accurate, this might be ok, but the word 'and' following the 'at' is a clear issue, I think at least the word 'and' right after word 'at' should be removed.

@jmrozanec
Copy link
Owner

@littleYanzi thank you for reporting this. We will be glad to merge a test with this issue if you submit the PR. Thanks!

@albertotn
Copy link
Contributor

I'd like to take care of this during hacktoberfest, could you assign me ?

@jmrozanec
Copy link
Owner

@albertotn sure! Thanks!

@edmeme
Copy link

edmeme commented Oct 2, 2020

There's another element missing in the description that is bothering me. It should read "Every second at 2 minutes and every 31 minutes" (and I'd say "every second at minute 2 and every 31 minutes" is more clear)

albertotn pushed a commit to albertotn/cron-utils that referenced this issue Oct 5, 2020
albertotn pushed a commit to albertotn/cron-utils that referenced this issue Oct 6, 2020
jmrozanec added a commit that referenced this issue Oct 6, 2020
jmrozanec added a commit that referenced this issue Oct 11, 2020
@jmrozanec jmrozanec added this to the next milestone Oct 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants