Skip to content

Add CI troubleshooting notes for Playwright browser setup #5

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

Open
harshvardhan60792 wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add CI troubleshooting notes for Playwright browser setup #5

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
5 changes: 5 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,11 @@ harness/ralph.sh 2 20 sh -c 'copilot --output-format json --stream on --allow-al
- Edit the check registry in [gate-data.ts](harness/gate-data.ts) and CI in [ci.yml](.github/workflows/ci.yml)
- `semgrep` uses registry configs (`p/typescript`, `p/javascript`, `p/security-audit`); it needs network and a machine-level install

## CI Troubleshooting

**Playwright browser mismatch in GitHub Actions**
If Playwright tests fail in CI because multiple browser projects are configured but only one browser is installed, run `pnpm --prefix harness run setup:e2e` in your CI workflow. This command installs the browser set expected by the Playwright config.

## Coordination

- Use `git log --oneline <branch>..HEAD` to show what's unpushed.
Expand Down