It is not a good idea for using node-schedule in a Puppet implementation.

[su-chang]

See detail here: #322 (comment)

• setTimeout
• node-schedule

[hcfw007]

If node-schedule package has no known issue, i think it's fine to use this package.

I mean there's no need to create a new wheel if the existing one is just fine, and there will possibly be more scheduled jobs in the future that could use node-schedule like clearing cache.

[huan]

After I quickly reviewed the code I can not see any necessity to use this pattern.

It only adds up the complexity and can be simply replaced by setTimeout with a few lines of code I think.

However, I'm ok if both of you feel better by using node-schedule.