You can clone with
HTTPS or Subversion.
The documentation referenced in the readme says that day of week is 1-7, Sunday being 1.
All other crontab documentation I see says that day of the week is 0-6, Sunday being 0.
Mind if I submit a patch to change? Or would you like a "compatibility mode" that explicitly requests real cron times?
I'd also like to support the standard 5-digit crontime syntax that doesn't include seconds. Mind if I submit a patch to default to "a b c d e f" to "0 a b c d e f" ?
Yes. I speak patches. :) I would like things to be consistent. Obviously both would break things so I'd upgrade the version and make it clear that there was a break in backwards compatibility. But, I'm ok with this so long as it makes the library more consistent.
When is this getting into the code? Currently, it's definitely inconsistent.
I haven't seen a patch for this yet. I'm not sure if anyone is actively working on it.
I can pick this up again at some point. @pixelfreak, are you more interested in the 5-digit crontime syntax or the day-of-week inconsistency?
More interested in the day-of-week inconsistency. Thanks!
Actually, I am curious why there exists two different cron format and which is better. For example, the linux cron we are all familiar with can't do seconds interval, but this one can. So maybe it's better to keep it that way?
Since linux cron is the standard, personally I think node-cron should be consistent with its syntax/functionality. This would mean changing day-of-week to be consistent. Since the seconds interval doesn't contradict any standard cron syntax, it should just be an optional extended syntax (only if 6 digits are included). Will prepare the patch and see if people need backwards compatibility.
Alrighty, pull request up at #41. There are some comments in there. Let me know what you think.
Completed with GH-41.