ci: Use sticky PR comments for Playwright report summaries#6909
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 3 Skipped Deployments
|
Docker builds report
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6909 +/- ##
=======================================
Coverage 98.34% 98.34%
=======================================
Files 1335 1335
Lines 49707 49707
=======================================
Hits 48882 48882
Misses 825 825 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
|
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
|
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Failed testsfirefox › tests/environment-permission-test.pw.ts › Environment Permission Tests › Environment-level permissions control access to features, identities, and segments @enterprise |
Use a shared header with append mode so all matrix jobs contribute to one comment instead of creating separate ones.
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)Details
Playwright Test Results (oss - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)Details
|
|
Likely needs more tweaking (all runs are appended to a single comment which will probably blow it up all the way to the body length limit for long-living PRs), but I'd rather merge this sooner and open a follow-up PR. |
docs/if required so people know about the feature.Changes
In this PR, we migrate Playwright E2E test report PR comments from
daun/playwright-report-summarydirect commenting tomarocchino/sticky-pull-request-comment. This ensures that on CI re-runs, the existing comment is updated in place rather than creating a new one each time, reducing notification spam.create-comment: falseondaun/playwright-report-summaryto generate the summary output onlymarocchino/sticky-pull-request-comment@v2with a stableheaderkey per test type and runnerHow did you test this code?
Will be validated by the next PR that triggers E2E tests in CI.