Skip to content

[#476] Add GitHub Actions deploy workflow#482

Merged
realproject7 merged 1 commit intomainfrom
task/476-vercel-deploy
Mar 24, 2026
Merged

[#476] Add GitHub Actions deploy workflow#482
realproject7 merged 1 commit intomainfrom
task/476-vercel-deploy

Conversation

@realproject7
Copy link
Copy Markdown
Owner

Summary

  • Adds .github/workflows/deploy-production.yml — deploys to Vercel on push to main or manual dispatch
  • Based on the Dropcast reference workflow with repo check changed to realproject7/plotlink
  • vercel.json with backfill cron already exists (committed in [P1-7] Cron Backfill #12)

Fixes #476

Operator items (already done per issue)

  • Vercel Ignored Build Step set to exit 0
  • GitHub secrets (VERCEL_TOKEN, VERCEL_ORG_ID, VERCEL_PROJECT_ID) added
  • production environment created

Test plan

  • Merge to main triggers the Actions workflow
  • Workflow pulls env, builds, and deploys to Vercel
  • Vercel auto-deploy does NOT trigger (exit 0)
  • Cron job fires every 5 minutes at /api/cron/backfill

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
plotlink Ignored Ignored Mar 24, 2026 7:57am

Request Review

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Workflow is clean — correct repo guard, concurrency group, cancel-in-progress: false (right choice for deploys), proper permissions scope, and good inline docs. No issues.

Copy link
Copy Markdown
Collaborator

@project7-interns project7-interns left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Summary

The deploy workflow is focused and matches the issue's requested GitHub Actions production deployment shape. The existing vercel.json cron config and backfill auth guard cover the remaining code-side requirements.

Findings

  • None.

Decision

Approve because the workflow is minimal, uses the correct repository gate, and aligns with the acceptance criteria for #476.

@realproject7 realproject7 merged commit d58468b into main Mar 24, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[#475] Vercel deployment + CI/CD setup

2 participants