You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@openwhisk.apache.org by GitBox <gi...@apache.org> on 2017/10/24 09:19:23 UTC

[GitHub] jthomas commented on issue #39: Alarms package documentation refers to node-cron rather than cron

jthomas commented on issue #39: Alarms package documentation refers to node-cron rather than cron
URL: https://github.com/apache/incubator-openwhisk-package-alarms/issues/39#issuecomment-338928149
 
 
   Looks like the `cron` package supports both formats. If fields are missing from the cron string, it assumes [defaults for those missing fields values](https://github.com/kelektiv/node-cron/blob/master/lib/cron.js#L64). 
   https://github.com/kelektiv/node-cron/blob/master/lib/cron.js#L309-L311
   
   Think this can be closed as not an issue? 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services