-
Notifications
You must be signed in to change notification settings - Fork 252
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
rules option ignored #672
Comments
Hi @edsnowden, thanks for this report, and for your code and logs which made it straightforward to understand the issue and reproduce the behaviour. Also, that is a very pleasant amount of memory. For HTML_CodeSniffer: I'm reasonably confident this is the currently intended behaviour of For axe: Are you comfortable with this behaviour, with the understanding it represents addition rather than replacement? Some other changes we could make:
|
This is surely a misunderstanding or error on my part, but as far as I can tell the 'rules' option is checked but ignored.
Expected behaviour
When a list of rules is given in the rules option, only those rules are checked for/reported.
Actual behaviour
Specified rules must be valid, but appear to have no effect.
Steps to reproduce
Install node and pa11y
Create a file bug.js:
Run with
Observe the output is the same whether the rules are specified or not, and the errors do not relate to the specified rule:
Environment
The text was updated successfully, but these errors were encountered: