ci(e2e): refresh apt before playwright --with-deps (ondrej PPA label change)#1902
ci(e2e): refresh apt before playwright --with-deps (ondrej PPA label change)#1902arifulhoque7 wants to merge 1 commit into
Conversation
…change)
setup-php adds the ondrej/php PPA; its InRelease Label changed from 'PPA
for PHP' to a redirect notice, so apt-get update fails with exit 100 and
'playwright install --with-deps' dies ('Failed to install browsers'). Run
apt-get update --allow-releaseinfo-change first to acknowledge the change.
Only runs on Playwright cache miss (when --with-deps actually hits apt).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughAdds a conditional step to the Playwright e2e GitHub Actions workflow that refreshes Ubuntu apt sources via ChangesE2E Workflow Apt Fix
Estimated code review effort: 1 (Trivial) | ~3 minutes Suggested labels: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
setup-php adds the ondrej/php PPA; its InRelease Label changed ("PPA for PHP" → redirect notice), so
apt-get updatefails with exit 100 andplaywright install --with-depsdies ("Failed to install browsers"). This blocked the e2e run right after the wpuf-pro clone succeeded.Fix: run
apt-get update --allow-releaseinfo-changebefore the browser-deps install, only on Playwright cache miss (when --with-deps actually hits apt).🤖 Generated with Claude Code
Summary by CodeRabbit