search

philipwalton / html-inspector

HTML Inspector is a code quality tool to help you and your team write better markup. It's written in JavaScript and runs in the browser, so testing your HTML has never been easier.
2.32k stars 145 forks source link

Add ability to exclude specific rule #22

Closed scottgallant closed 11 years ago

scottgallant commented 11 years ago

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

philipwalton commented 11 years ago

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 commented 11 years ago

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 commented 11 years ago

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 commented 11 years ago

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

philipwalton commented 11 years ago

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 commented 11 years ago

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

scottgallant commented 11 years ago

Very glad to see this! Thanks!