pa11y / pa11y-ci

Pa11y CI is a CI-centric accessibility test runner, built using Pa11y
https://pa11y.org
GNU Lesser General Public License v3.0
515 stars 63 forks source link

No --reporter flag available #163

Closed greggles closed 2 years ago

greggles commented 2 years ago

Hello,

I'm having trouble using the --reporter flag. here's my version, config file, and the command failing when I try to specify a reporter and succeeding if I leave off the reporter. Any tips? Thanks!

greggles@gregs-mbp ~/c/a/a11y (main) > pa11y-ci -V
2.4.2
greggles@gregs-mbp ~/c/a/a11y (main)> cat .pa11yci
{
    "defaults": {
        "reporters": [
            "cli",
            ["json", { "fileName": "./results.json" }]
        ]
    },
    "urls": [
        "https://pa11y.org/"
    ]
}
greggles@gregs-mbp ~/c/a/a11y (main)> pa11y-ci --reporter=json
error: unknown option `--reporter'
greggles@gregs-mbp ~/c/a/a11y (main) [1]> pa11y-ci
Running Pa11y on 1 URLs:
 > https://pa11y.org/ - 0 errors

✔ 1/1 URLs passed
greggles@gregs-mbp ~/c/a/a11y (main)>
joeyciechanowicz commented 2 years ago

Apologies, the --reporters flag is there, but not on the currently released version of pa11y-ci. If you grab the source and run it directly, then it should work no problem.

greggles commented 2 years ago

Got it, OK. And I realized in the older versions there is a -j option which gets me what I need so I'll close this.

For what it's worth, I do think it could be good to create a new version or use a branching strategy that lets the README on github match the latest stable version.