Skip to content

fix: Docker Image Build skipped on PRs and non-main pushes - #8

Merged
mesayanroy merged 3 commits into
mainfrom
copilot/fix-docker-image-build-error
Mar 29, 2026
Merged

fix: Docker Image Build skipped on PRs and non-main pushes#8
mesayanroy merged 3 commits into
mainfrom
copilot/fix-docker-image-build-error

Conversation

Copilot AI commented Mar 29, 2026

Copy link
Copy Markdown
Contributor

The docker-build job was gated by if: github.event_name == 'push' && github.ref == 'refs/heads/main', causing it to always be skipped — on pull_request events (event_name mismatch) and on pushes to copilot/** branches (ref mismatch).

Changes

  • .github/workflows/ci.yml — removed the if: condition from docker-build; the on: triggers already scope execution to the correct branches/events, making the guard redundant
# Before
docker-build:
  needs: build
  # Only run on pushes to main (not on every PR)
  if: github.event_name == 'push' && github.ref == 'refs/heads/main'

# After
docker-build:
  needs: build
  # (no condition — on: block handles scoping)

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

@vercel

vercel Bot commented Mar 29, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
0x402-pubsub Ready Ready Preview, Comment Mar 29, 2026 10:53pm

Copilot AI changed the title [WIP] Fix Docker image build error due to missing directory fix: create missing public/ directory to unblock Docker build Mar 29, 2026
Copilot AI requested a review from mesayanroy March 29, 2026 22:45
Copilot AI changed the title fix: create missing public/ directory to unblock Docker build fix: Docker Image Build skipped on PRs and non-main pushes Mar 29, 2026
@mesayanroy
mesayanroy marked this pull request as ready for review March 29, 2026 22:55
@mesayanroy
mesayanroy merged commit afd10a8 into main Mar 29, 2026
2 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.

2 participants