Skip to content

Fix CI/CD pipeline with deploy job#35

Merged
vimal-java-dev merged 1 commit intovimal-java-dev:mainfrom
vimal-tech-dev:feature/new-update-phase-10
Apr 5, 2026
Merged

Fix CI/CD pipeline with deploy job#35
vimal-java-dev merged 1 commit intovimal-java-dev:mainfrom
vimal-tech-dev:feature/new-update-phase-10

Conversation

@vimal-tech-dev
Copy link
Copy Markdown
Contributor

Deployment added with this condition in the yml file,

deploy:
# ✅ CRITICAL CONDITION (owner repo only)
if: github.event_name == 'push' && github.repository == 'vimal-java-dev/vimaltech-contact-api'
needs: docker
runs-on: ubuntu-latest

Signed-off-by: vimal-tech-dev <be08it929@gmail.com>
@vimal-java-dev vimal-java-dev added enhancement New feature or request help wanted Extra attention is needed question Further information is requested labels Apr 5, 2026
- name: Push Docker image
run: |
docker push ghcr.io/vimal-java-dev/vimaltech-contact-api:latest No newline at end of file
docker push ghcr.io/vimal-java-dev/vimaltech-contact-api:latest
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Good for auto-deploy to VPS

✅ 3. Proper Dependency Chain

deploy → needs: docker
docker → needs: build

Because,
👉 Guarantees:

  • No deploy if build fails
  • No deploy if image push fails

@vimal-java-dev vimal-java-dev merged commit 9bff2f3 into vimal-java-dev:main Apr 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request help wanted Extra attention is needed question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants