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 a11y tests to workflow #641
base: main
Are you sure you want to change the base?
Add a11y tests to workflow #641
Conversation
Signed-off-by: Matt Provost <provomat@amazon.com>
Signed-off-by: Matt Provost <provomat@amazon.com>
Accessibility errors will go away when #640 is merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ddint see the test failures before i approved them, can we also fix the a11y issues before adding them to the CI? No point adding something to the CI thats broken, making the CI useless.
Converting to draft until we get guidance on how to fix a11y issues |
Description
Run accessibility tests when running the build and test workflow. The tests are and have been in the repository for a while, not sure why they weren't included in the workflow
Check List
yarn lint
yarn test-unit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.