Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alarms package documentation refers to node-cron rather than cron #1769

Closed
hassenius opened this issue Jan 27, 2017 · 2 comments
Closed

Alarms package documentation refers to node-cron rather than cron #1769

hassenius opened this issue Jan 27, 2017 · 2 comments

Comments

@hassenius
Copy link
Contributor

The documentation for the alarms package refers to the format of node-cron (for example second field is optional)
https://github.com/openwhisk/openwhisk/blob/master/docs/catalog.md#firing-a-trigger-event-periodically

However, looking at the source, alarms uses the cron package:
https://github.com/openwhisk/openwhisk-package-alarms/blob/master/package.json
Which seems to require 6 fields, etc

Cron package documentation
https://www.npmjs.com/package/cron

@rabbah
Copy link
Member

rabbah commented Feb 24, 2017

@csantanapr @jasonpet move?

@markusthoemmes
Copy link
Contributor

Issue moved to openwhisk/openwhisk-package-alarms #39 via ZenHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants