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

Add JSDoc to devDependencies; document RedisClient #167

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

mbland
Copy link
Owner

@mbland mbland commented Dec 9, 2017

This is the first step towards properly documenting all of the classes that comprise the application.

cc: @mpdatx (I'll add you as a proper reviewer once you've accepted the collaborator invite.)

This is the first step towards properly documenting all of the classes
that comprise the application.
@mbland mbland added this to the v1.0.0 milestone Dec 9, 2017
@mbland mbland self-assigned this Dec 9, 2017
@mbland mbland requested a review from akashvbabu December 9, 2017 18:54
@coveralls
Copy link

coveralls commented Dec 9, 2017

Coverage Status

Coverage remained the same at 98.061% when pulling ab4b973 on jsdoc into 42960b6 on master.

It's now possible to run `./go jsdoc -h` and `./go jsdoc -d
<output-dir>` and have it do the right thing.
Also prevents `scripts/lint` from passing JSDoc-generated files to
ESLint.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.061% when pulling 4e3c871 on jsdoc into 42960b6 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 98.061% when pulling 4e3c871 on jsdoc into 42960b6 on master.

@mbland mbland mentioned this pull request Jan 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants