Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Pull Request Overview
This PR updates the GitHub Actions workflows for Playwright to fetch more commit history during checkout, ensuring plugins that rely on recent commit data work correctly.
- Increase
fetch-depthfrom the default 1 to 2 for Git checkout in both workflows. - Applies the same change to the main Playwright workflow and the Cloudflare Open Next workflow.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/playwright.yml | Added fetch-depth: 2 under the checkout step |
| .github/workflows/playwright-cloudflare-open-next.yml | Added fetch-depth: 2 under the checkout step |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #7832 +/- ##
==========================================
- Coverage 75.47% 75.46% -0.02%
==========================================
Files 101 101
Lines 8309 8309
Branches 218 218
==========================================
- Hits 6271 6270 -1
- Misses 2036 2037 +1
Partials 2 2 ☔ View full report in Codecov by Sentry. |
AugustinMauroy
left a comment
There was a problem hiding this comment.
LGTM ! That's work but I don't get why we need to increase fetch dep. I mean why limit as been reach.
git fetch depth of 2 allows git to run comparisons; fetch dept: 1 is current commit, 2 is the bare minimum for git to be able to make a diff. |
|
BTW, @avivkeller feel free to fast track this, you have my +1. |
|
Lighthouse Results
|
Ref: #7830 (comment) (https://github.com/microsoft/playwright/blob/main/packages/playwright/src/plugins/gitCommitInfoPlugin.ts#L74)