-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update documentation #23
Conversation
2f9769e
to
a4af197
Compare
Seegno-flavored JSCS config. | ||
|
||
## Installation | ||
|
||
```sh | ||
$ npm install jscs jscs-config-seegno --save-dev | ||
$ npm install jscs jscs-preset-seegno should --save-dev |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would move the preset rename to another PR so we can rename the repo + package too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense.
a4af197
to
78bf2f0
Compare
The preset includes the following custom rules: | ||
|
||
### `disallowGeneratorsInDescribeFunctions` | ||
Disallows the usage of generators for the `describe` grouping primitive that features in testing frameworks such as `mocha`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you include a small paragraph on why this is critical to validate?
Looks good minor small comments. |
78bf2f0
to
b6c4d9a
Compare
@ruimarinho I've added a small explanation for each rule. |
LGTM. |
No description provided.