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

[Issue #677]: Add pa11y accessibility checks against PR preview env #678

Closed
wants to merge 4 commits into from

Conversation

rylew1
Copy link
Contributor

@rylew1 rylew1 commented Jul 1, 2024

Ticket

Resolves #677

Changes

  • add pa11y ci checks

@rylew1 rylew1 requested a review from lorenyu July 1, 2024 18:53
@lorenyu
Copy link
Contributor

lorenyu commented Jul 1, 2024

@rylew1 take a look at the template development workflow can you test out the changes on either the platform-test repo or the platform-test-nextjs repo?

@rylew1 rylew1 closed this Jul 16, 2024
@rylew1
Copy link
Contributor Author

rylew1 commented Jul 16, 2024

Closing this - we ended up steering away from a11y in favor of e2e playwright a11y tests - the more built out pa11y PR is here: navapbc/platform-test-nextjs#84

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-ci accessibility checks against PR preview env
2 participants