Skip to content
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

WIP: Pa11y testing to github actions #162

Closed
wants to merge 3 commits into from
Closed

Conversation

jhoude5
Copy link
Contributor

@jhoude5 jhoude5 commented Jul 12, 2021

No description provided.

@github-actions
Copy link

Visit the preview URL for this PR (updated for commit 4044f7a):

https://ca-it-prod-home-5918--pr162-jennifer-pa11y-tests-bo9ys5o4.web.app

(expires Mon, 19 Jul 2021 18:28:12 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@jhoude5 jhoude5 changed the title adding sitemap.xml to site WIP: Pa11y testing to github actions Jul 12, 2021
@jhoude5 jhoude5 linked an issue Jul 12, 2021 that may be closed by this pull request
@grugnog
Copy link
Member

grugnog commented Jun 24, 2022

Closing this, since we can't run pa11y on the site before the site is deployed - potentially we could extend #198 to do this, but lighthouse runs axe checks also so it may not be necessary unless we need the reporting that pa11y provides.

@grugnog grugnog closed this Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add pa11y testing to PR's
2 participants