Skip to content

Scan the pages a PR changes with axe - #2264

Draft
cwickham wants to merge 5 commits into
mainfrom
ci/axe-pr-check
Draft

cwickham wants to merge 5 commits into
mainfrom
ci/axe-pr-check

Conversation

@cwickham

@cwickham cwickham commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Adds an axe job to the preview workflow. It reuses the preview render and runs quarto call axe on the pages the PR changes. It fails on new serious or critical findings that are not in _axe-baseline.json.

  • A change that can reach every page (SCSS, _quarto.yml, shared partials, extensions, the baseline) scans the whole site.
  • Pages in _axe-exclude.txt are skipped.
  • The site artifact uploads after the PR comment, so the preview links are not delayed.

Still to test: one PR that changes a single page, and one that changes SCSS (full-site scan).

- Scan only the changed pages that rendered in _site
- Read the exclude list from _axe-exclude.txt
- Upload the site artifact after the PR comment, with overwrite: true
- Skip the job summary if the scan wrote no report
- Do not start a full-site scan for _axe-report.md
@github-actions

github-actions Bot commented Oct 2, 2026

Copy link
Copy Markdown
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-2264.quarto.org

This branch has not been deployed

No deployments
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.

1 participant