[Issue #347]: Add pa11y-ci checks

navapbc / template-application-nextjs

Next.js, TypeScript, USWDS, and Storybook template, including CI/CD, for teams at Nava building web applications

Apache License 2.0

10 stars 2 forks source link

[Issue #347]: Add pa11y-ci checks #350

Open rylew1 opened 3 weeks ago

rylew1 commented 3 weeks ago

Ticket

Resolves #347

Changes

add pa11y-ci package
Add PR checks for mobile and desktop against the dynamically generated sitemap

context

Not sure if we want to document this in a decision record?
localhost:3000/sitemap.xml

github-actions[bot] commented 3 weeks ago

Coverage report for `app`

St.^{:grey_question:}	Category	Percentage	Covered / Total
🟢	Statements	93.1%	81/87
🟢	Branches	82.35%	14/17
🟢	Functions	93.33%	14/15
🟢	Lines	93.59%	73/78

Test suite run success

16 tests passing in 5 suites.

Report generated by 🧪jest coverage report action from 3d2149c9e29c3b72a4d8df338c6032f7bfa81eda

aligg commented 2 weeks ago

If we don't add a decision record it could be nice to at least update the README to explain what pa11y-ci is, how to run it, what it's checking for, in brief.