chore: more verbose Netlify deploy logs - #17057
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
Install the latest version of pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/7ec20d47903b5be12c26fb0249eb85ec508695b3Open in |
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: QUIET Plan: Advanced Run ID: 📒 Files selected for processing (1)
🔗 Linked repositories identifiedCodeRabbit considers these linked repositories for cross-repo context during reviews:
Included review availability: Your plan provides up to 10 included reviews per hour; 6 remain after this review. 📝 WalkthroughWalkthroughThe Netlify action now permits Merge Risk: 🔵 Low · up to Malformed Netlify CLI output can make a deployment run fail without the intended error details, leaving platform-test failures harder to diagnose. This is a bounded CI observability risk. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
Note
Quiet mode is enabled, so only the most important comments were posted inline. Other review comments are grouped below.
🟡 Other comments (1)
.github/actions/netlify-deploy/action.yml-49-50 (1)
49-50: 🩺 Stability & Availability | 🟡 Minor | ⚡ Quick winHandle invalid deployment output explicitly.
If
.deploy_urlis absent ornull,jq -efails at the assignment on line 49. Withset -e, the action exits before it reports the failure or printsdeploy_json.jq -ealso accepts an empty string. Wrap the extraction in anifstatement and validate the URL before writing it toGITHUB_OUTPUT.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository YAML (base), Organization UI (inherited)
Review profile: QUIET
Plan: Advanced
Run ID: b278afb2-1233-442e-8675-033476181596
📒 Files selected for processing (1)
.github/actions/netlify-deploy/action.yml
🔗 Linked repositories identified
CodeRabbit considers these linked repositories for cross-repo context during reviews:
sveltejs/vite-plugin-svelte(manual)vitejs/vite(manual)sveltejs/svelte(manual)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
same changes already made on the
mainbranch in #17052makes deploy logs a bit more verbose