-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add support for aXe runner #88
Comments
I would have thought allowing specifying the runner(s) would be preferable, but would take longer - it would allow separation if necessary for results, as well as setting a customisable model for if/when future runners may be added. I guess we would also need to set/decide what to do about existing tests that have no runner specified - would they maintain HTMLCS as the default or default to HTMLCS and aXe? Thinking about pa11y-dashboard, the latter may cause unintentional changes to comparing historical data if relying on the graph tracking, but defaulting to all available runners makes migrating jobs to run on all runners more straightforward as I guess most would want to run on all available runners. Just my thoughts! |
I cannot find the code where it specifies the runner parameter, could you please point it out? I ask because pa11y documentation states that runner parameter is needed, else it'll throw an error |
Hello, the default runner is |
@sangitamane Thank You! I checked it out. I was confused with the pa11y documentation, but now it's cleared. |
Pa11y (not currently published) now supports multiple test runners. We need to add the ability to select runners in the webservices.
Therse a couple of ways this could be done:
The text was updated successfully, but these errors were encountered: