Cron.matchDate returns always true when using weekdays and days of month about cron-schedule HOT 3 CLOSED

Hello @renzo-s 👋

Thank you for your feedback. The expression 0-6 in the weekdays field will not result in only Saturdays or sundays, but will allow any weekday between 0 (Sunday) and 6 (Saturday), which includes 1 (Monday), 2 (Tuesday), …, which at the end is every weekday.
If you only want to include Sunday and Saturday, use 0,6 instead (comma instead of dash).

The logic that either day of month or weekday needs to match, comes from this note from crontab.org:

Note: The day of a command's execution can be specified by two fields -- day of month, and day of week. If both fields are restricted (ie, aren't *), the command will be run when either field matches the current time.

So you are right, that this check is wrong, when one of the fields is not restricted.

from cron-schedule.

Hi Pascal, 🙈 damn that was a silly typo. I'm using 0,6 not 0-6 (edited now). Sorry for the confusion. Thanks for the quick reply.

Ah, thanks for the spec note, now the brackets and the OR make sense. This just needs another * check 👌

from cron-schedule.

🎉 This issue has been resolved in version 3.0.6 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

from cron-schedule.