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

Update README with explanation recommended/strict #238

Merged
merged 3 commits into from
May 11, 2017

Conversation

Donaldini
Copy link
Contributor

Added section to readme that explains the difference between recommended and strict rule configuration.

Added section to readme that explains the difference between recommended and strict rule configuration.
@coveralls
Copy link

coveralls commented May 10, 2017

Coverage Status

Coverage remained the same at 99.547% when pulling cf8d09f on Donaldini:patch-1 into c156644 on evcohen:master.

Added strict to the examples of enabling a mode.
@coveralls
Copy link

coveralls commented May 11, 2017

Coverage Status

Coverage remained the same at 99.547% when pulling bbe1d19 on Donaldini:patch-1 into c156644 on evcohen:master.

README.md Outdated
@@ -89,7 +90,130 @@ You can also enable all the recommended rules at once. Add `plugin:jsx-a11y/reco
}
```

### Difference between 'recommended' and 'strict' mode
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great! thank you!! can we move this below the list of supported rules, though?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@evcohen Sure, I was doubting whether to put it under or above myself :)
Fixed it.

@coveralls
Copy link

coveralls commented May 11, 2017

Coverage Status

Coverage remained the same at 99.547% when pulling 52270da on Donaldini:patch-1 into c156644 on evcohen:master.

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.

None yet

4 participants