Skip to content

fix(workflows): use SCANNER_URL secret in scan workflows and add deploy summary AB#2137#49

Merged
emmanuelknafo merged 1 commit intomainfrom
feature/2137-fix-scanner-url-and-deploy-summary
Mar 29, 2026
Merged

fix(workflows): use SCANNER_URL secret in scan workflows and add deploy summary AB#2137#49
emmanuelknafo merged 1 commit intomainfrom
feature/2137-fix-scanner-url-and-deploy-summary

Conversation

@emmanuelknafo
Copy link
Copy Markdown
Contributor

@emmanuelknafo emmanuelknafo commented Mar 29, 2026

Summary

Two fixes:

  1. Scan-all fails with exit code 6 — all 5 demo app a11y-scan.yml workflows had a hardcoded scanner URL (a11y-scan-demo-app.azurewebsites.net) that doesn't match the actual deployed URL (which includes a uniqueString suffix). Changed to use SCANNER_URL secret.
  2. Deploy-all has no clickable links — added a deployment summary job that queries Azure for all 6 app URLs and renders them as clickable links in the GitHub Actions step summary.

Changes

Scan fix

  • a11y-demo-app-{001-005}/.github/workflows/a11y-scan.yml — replace hardcoded URL with secrets.SCANNER_URL
  • scripts/bootstrap-demo-apps.ps1 — auto-resolve scanner URL from Azure deployment, set as SCANNER_URL secret on each demo repo
  • Manually set SCANNER_URL secret on all 5 sibling repos

Deploy summary

  • deploy-all.yml — add summary job that prints a markdown table with clickable links for all 5 demo apps and the scanner app

Fixes AB#2137

…oy summary AB#2137

- replace hardcoded scanner URL with SCANNER_URL secret in all 5 demo apps
- add deployment summary job to deploy-all with clickable links for all apps
- add SCANNER_URL secret provisioning to bootstrap script

🔧 - Generated by Copilot
@github-actions
Copy link
Copy Markdown

Coverage Report for Coverage

Status Category Percentage Covered / Total
🔵 Lines 83.29% (🎯 80%) 723 / 868
🔵 Statements 82.82% (🎯 80%) 786 / 949
🔵 Functions 80.86% (🎯 80%) 131 / 162
🔵 Branches 70.09% (🎯 65%) 368 / 525
File CoverageNo changed files found.
Generated in workflow #131 for commit 61c5401 by the Vitest Coverage Report Action

@emmanuelknafo emmanuelknafo merged commit 914c2f7 into main Mar 29, 2026
14 of 18 checks passed
@emmanuelknafo emmanuelknafo deleted the feature/2137-fix-scanner-url-and-deploy-summary branch March 29, 2026 22:28
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