-
Notifications
You must be signed in to change notification settings - Fork 65
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 JavaScript config example to README #197
Add JavaScript config example to README #197
Conversation
The GitHub actions failures seem to be caused by |
2b1352e
to
c8d809c
Compare
a1dfb9f
to
ef3f7b6
Compare
@aarongoldenthal Hello again! Thanks for this contribution too. As I mentioned I'd like to merge it into P.S. About your pinning comment - as you've probably noticed, I have settled for the pinning for now, for these minor-level updates, although I'm working to describe the |
@aarongoldenthal Hello again! Thanks for this contribution too. As I mentioned I'd like to merge it into @danyalaytekin LGTM |
* origin/master: Bump ip from 1.1.8 to 1.1.9 Update to Pa11y v7 and other dependency updates (pa11y#231) Version `3.1.0` (pa11y#226) Add JavaScript config example to README (pa11y#197) Bump @babel/traverse from 7.16.0 to 7.23.3 (pa11y#222) Bump semver from 6.3.0 to 6.3.1 (pa11y#221) Bump json5 from 2.2.0 to 2.2.3 (pa11y#219) Bump ansi-regex from 3.0.0 to 3.0.1 (pa11y#218) Bump debug from 4.1.1 to 4.3.1 (pa11y#216) Bump node-fetch from 2.6.1 to 2.6.7 (pa11y#217) Upgrade to `pa11y@6.2.3`, add publishing workflow (pa11y#213) Update integration tests to support all platforms, including Windows (pa11y#177) # Conflicts: # bin/pa11y-ci.js # package-lock.json # package.json
Update README to include additional details and example with JavaScript config file, including how it can be used to dynamically update config. Based on question from #193.