fix(ci): auto-merge does not need checkout#8708
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/web-infra Please review the changes when you have a chance. Thank you! 🙏 |
There was a problem hiding this comment.
Pull request overview
Removes an unnecessary repository checkout from the scheduled auto-merge workflow, aligning it with how the workflow is used here (no local action references needed).
Changes:
- Delete the
actions/checkoutstep from the auto-merge workflow job.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8708 +/- ##
==========================================
+ Coverage 75.09% 75.10% +0.01%
==========================================
Files 104 104
Lines 9161 9161
Branches 315 315
==========================================
+ Hits 6879 6880 +1
+ Misses 2280 2279 -1
Partials 2 2 ☔ View full report in Codecov by Sentry. |
📦 Build Size ComparisonSummary
Changes➕ Added Assets (1)
➖ Removed Assets (1)
|
Description
I realise now I left the checkout step in #8701, which is not required (unlike in web-team) as we're not referencing a local action.
Validation
N/A
Related Issues
N/A
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.