Skip to content

ci: drop Docker build/push job from deploy workflow - #73

Merged
Programcode merged 1 commit into
mainfrom
codex/strip-docker-job
Sep 27, 2026
Merged

Programcode merged 1 commit into
mainfrom
codex/strip-docker-job

Conversation

@Programcode

@Programcode Programcode commented Sep 27, 2026 •

Copy link
Copy Markdown
Owner

Deployment target is Vercel now and nothing consumes the adamgaborjaniszabo/debate-trainer image anymore (verified: no references on the VPS), so the docker job and the production-assets artifact handoff that only fed it are removed.

Keeps the quality, database-tests, and browser-tests gates untouched. Workflow name intentionally left as-is: staging-e2e.yml triggers on it via workflow_run.

Summary by CodeRabbit

  • Chores
    • Removed the automated Docker image build and publishing workflow. Production build artifacts are no longer uploaded by the deployment workflow.

Deployment target is Vercel, and nothing consumes the
adamgaborjaniszabo/debate-trainer image anymore, so the docker job
and the production-assets artifact handoff that only fed it are dead
weight. Keeps the quality, database-tests, and browser-tests gates.
Copilot AI lite review requested due to automatic review settings September 27, 2026 11:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Programcode has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@vercel

vercel Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
debate-navigator Ready Ready Preview Sep 27, 2026 11:57am UTC

@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: 1be8a7aa-4792-481b-9b27-0ac69f88b89c

📥 Commits

Reviewing files that changed from the base of the PR and between 76e1869 and c6e0b8a.

📒 Files selected for processing (1)
  • .github/workflows/deploy.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/deploy.yml

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The deployment workflow no longer uploads production assets as an artifact or runs the Docker job that built and pushed multi-architecture images.

Changes

Deployment workflow

Layer / File(s) Summary
Remove artifact upload and Docker publishing
.github/workflows/deploy.yml
The workflow no longer uploads the production build as an artifact. It also removes the Docker job that built and pushed the latest image for linux/amd64 and linux/arm64.

Estimated code review effort: 2 (Simple) | ~8 minutes


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Essentials by visiting https://app.coderabbit.ai/settings/billing.

Comment @coderabbitai help to get the list of available commands.

@Programcode
Programcode merged commit 0bda733 into main Sep 27, 2026
5 of 8 checks passed

This branch was successfully deployed

1 active deployment
Preview — c6e0b8ac Deployed Sep 27, 2026 by vercel[bot]
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.

2 participants