-
Notifications
You must be signed in to change notification settings - Fork 0
ci: implement prebuilt deploy with GitHub-linked deployments and hidd… #14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
WalkthroughThe CI workflow in .github/workflows/ci-deploy.yml was simplified: renamed job, consolidated steps to install/setup, streamlined Vercel config and deploy logic with inline environment detection, added URL parsing, updated PR comment and deployment-card steps, and restricted production deployment/card creation to the main branch. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor Dev as Developer
participant GH as GitHub Actions
participant VC as Vercel CLI
participant PR as GitHub PR
Dev->>GH: push / pull_request
activate GH
GH->>GH: Install dependencies (with CI fallback)
GH->>GH: Setup Vercel CLI
Note right of GH: Env inferred inline from branch (main => production)
rect rgba(200,235,255,0.3)
GH->>GH: Prebuild project (produce output)
GH->>VC: Deploy prebuilt output<br/>(--prod on main)
VC-->>GH: Deployment output (raw logs)
end
GH->>GH: Parse deployment URL from Vercel output
alt On pull_request
GH->>PR: Comment with bold preview URL
end
alt On main branch
GH->>GH: Create deployment card<br/>(environment: production)
end
deactivate GH
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
.github/workflows/ci-deploy.yml (1)
16-16: Normalize YAML spacing to satisfy lint warning.YAMLlint is flagging the double spaces after the colons on Line 16 (
VERCEL_ORG) and Line 59 (VC_TEAM). Trimming those extra spaces will keep the workflow lint-clean. Based on static analysis hints.Also applies to: 59-59
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
.github/workflows/ci-deploy.yml(1 hunks)
🧰 Additional context used
🪛 YAMLlint (1.37.1)
.github/workflows/ci-deploy.yml
[warning] 16-16: too many spaces after colon
(colons)
[warning] 59-59: too many spaces after colon
(colons)
…en prod URL
Summary by CodeRabbit
Chores
Refactor