Day of week number can't be bigger than 5

[dalex-am]

cron-validate/src/helper.ts

Line 151 in b6963fc

if (!occurrence || isNaN(occurrenceNum)) {

Can you add one more condition into "if"?

[Airfooox]

I don't quite know what you mean. The dayof the week can't be greater than 5? Why not?

Do you mean it shouldn't get greater than 5 and cron-validate doesnt check for it or doesn't it currently allow it to be bigger than 5?

If we look here for example: http://crontab.org/ or https://docs.aws.amazon.com/de_de/AmazonCloudWatch/latest/events/ScheduledEvents.html the day of the week can go from 0 to 7 / 1 to 7.

[dalex-am]

I mean value "1#8" equals to 8th sunday/monday of month. Max count of any DOW cannot be greater than 5 just because of calendar structure. Write here "* * * ? * 3#8 *"

[Airfooox]

Alright, thanks. I'm gonna check that as soon as I got time to do so (like Sunday/Monday)

[Airfooox]

Fixed this in version 1.4.5, thanks bringing up the issue!