Skip to content
This repository has been archived by the owner on Dec 7, 2017. It is now read-only.

Add ability to exclude specific rule #22

Closed
scottgallant opened this issue Jul 23, 2013 · 7 comments
Closed

Add ability to exclude specific rule #22

scottgallant opened this issue Jul 23, 2013 · 7 comments

Comments

@scottgallant
Copy link

It would be great to be able to exclude rules as well. Also, a list of default rules would be handy too.

@philipwalton
Copy link
Owner

Yeah, excluding rules is on my list of TODOs before the next release. I'll track it via this issue.

What do you mean about a list of default rules? Like in the code or just documented more clearly somewhere?

@scottgallant
Copy link
Author

Glad to hear it's on your radar!

Regarding the "default rules", I was just looking for a list of rules (such as the "unused-classes" rule) to add to useRules config option. I didn't see a complete list in the docs. Thanks!

@philipwalton
Copy link
Owner

The README does contain a complete list of the built-in rules, though they are separated into categories so maybe it's not super easy to get a holistic overview.

@scottgallant
Copy link
Author

Is the config-friendly name for each rule the lowercase, hyphenated name? For example, "Script Placement" would become "script-placement"?

@philipwalton
Copy link
Owner

Ahh, OK, now I get what your question is.

Yeah, I can make it more clear what the name identifier is for each of the default rules.

@scottgallant
Copy link
Author

Haha, sorry for the confusion. Looking forward to the updates!

@scottgallant
Copy link
Author

Very glad to see this! Thanks!

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

No branches or pull requests

2 participants