Skip to content

test a11y test as warn not fail #1083

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.

Sign up for GitHub

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

Merged
birm merged 3 commits into from
Feb 18, 2025
Merged

test a11y test as warn not fail #1083

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
c44a717
test a11y test as warn not fail
birm Feb 18, 2025
ec85b22
try to get explicit warning message
birm Feb 18, 2025
67ced4b
different warning syntax, no set status
birm Feb 18, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion .github/workflows/axe-a11y-check.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
name: axe

on:
push:
branches:
Expand All @@ -8,6 +9,7 @@ on:
branches:
- master
- develop

jobs:
axe:
runs-on: ubuntu-latest
Expand All @@ -27,7 +29,12 @@ jobs:
- name: Install specific version of ChromeDriver
run: npm install -g chromedriver@125
- name: Run axe
id: axe-test
run: |
npm install -g @axe-core/cli
sleep 90
axe http://127.0.0.1:8080 --chromedriver-path $(npm root -g)/chromedriver/bin/chromedriver --exit
axe http://127.0.0.1:8080 --chromedriver-path $(npm root -g)/chromedriver/bin/chromedriver --exit || echo "status=warning" >> $GITHUB_ENV

- name: Set output status to warning (if test fails)
if: env.status == 'warning'
run: echo "::warning::Accessibility tests failed, but this does not block (most) PRs."
Loading