fix(ci): keep official actions/checkout in Depot release-assets workflow - #520
Merged
Merged
Conversation
Depot CI replaces a bare `actions/checkout` (no inputs) with its own shallow-pack clone, which leaves no `origin` remote. create-pull-request reads remote.origin.url and failed with "The format of '' is not a valid GitHub repository URL" on every run that had assets to commit. Passing an explicit `fetch-depth: 1` keeps the official action, which sets up `origin` normally. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Every Depot run of Update Arcane Release Assets that actually had assets to commit failed at the Create pull request step:
Depot CI silently replaces a bare
actions/checkoutstep (nowith:inputs) with its own shallow-pack clone script. That script never adds anoriginremote, andpeter-evans/create-pull-requestreadsremote.origin.urlto find the repo and push.It only ever worked before because the cron ran on GitHub Actions, where
actions/checkoutis real. Once releases were routed to the Depot copy on Sep 15, the first run with real changes (Sep 16) broke.Fix
Give the first checkout an explicit
fetch-depth: 1(the default value) and bump toactions/checkout@v7. Any explicit input keeps Depot from substituting its script, so the official action runs andoriginis set as usual.Verification
Probed on Depot: bare checkout → no origin;
fetch-depth: 1/fetch-depth: 0→ real action, origin = github.com. Then ran this workflow end to end on Depot with the change; it opened #519 successfully.🤖 Generated with Claude Code