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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved readme #740

Merged
merged 3 commits into from Dec 17, 2018

Conversation

Projects
None yet
2 participants
@noygal
Copy link
Contributor

noygal commented Nov 29, 2018

The README.md was missing the default value for processEvery, had to look it up on the source code, link: https://github.com/agenda/agenda/blob/master/lib/agenda/index.js#L33

Hope this PR would help someone else in the future, keep on with the great work 馃憤

noygal and others added some commits Nov 29, 2018

Merge pull request #1 from noygal/noygal-patch-1
Docs - added `processEvery` defaults to README.md
@vkarpov15
Copy link
Collaborator

vkarpov15 left a comment

Made a quick fix but LGTM 馃憤

@vkarpov15 vkarpov15 merged commit b1f2cbb into agenda:master Dec 17, 2018

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
security/snyk - package.json (agenda) No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment