Skip to content

feat: update CI pipeline to include testing and improve Docker build … - #2

Merged
youssef-ar merged 1 commit into
mainfrom
feat/cicd
Apr 6, 2026
Merged

youssef-ar merged 1 commit into
mainfrom
feat/cicd

Conversation

@youssef-ar

@youssef-ar youssef-ar commented Apr 6, 2026 •

Copy link
Copy Markdown
Contributor

…process

Summary by CodeRabbit

Release Notes

  • Chores
    • Enhanced CI/CD pipeline with automated test execution and artifact capture for improved build reliability.
    • Optimized Docker image build process with refined versioning and deployment strategy.

@coderabbitai

coderabbitai Bot commented Apr 6, 2026 •

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 307989a9-8536-4f7d-bd1b-33e5ad0aa385

📥 Commits

Reviewing files that changed from the base of the PR and between e51d1b3 and 55fe7bc.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

Activated a test job running Maven tests with artifact collection. Updated build job to depend on both compile and test jobs. Reworked Docker image building to tag PR builds with Git SHA without pushing, and only login/push to Docker Hub on main branch push events.

Changes

Cohort / File(s) Summary
CI/CD Workflow Refactoring
.github/workflows/ci.yml
Activated test job with artifact collection, updated build job dependencies to wait for both compile and test, and reworked Docker image tagging strategy to use Git SHA for PRs with conditional Docker Hub login/push on main branch pushes only.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 Thump thump! Tests now run with care,
Building, testing, Docker everywhere!
No more skipping—tests lead the way,
SHA tags for PRs, main branch to stay,
A fluffy pipeline, quick and fair!


Note

🎁 Summarized by CodeRabbit Free

The PR author is not assigned a seat. To perform a comprehensive line-by-line review, please assign a seat to the pull request author through the subscription management page by visiting https://app.coderabbit.ai/login.

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

@sonarqubecloud

sonarqubecloud Bot commented Apr 6, 2026

Copy link
Copy Markdown

@youssef-ar
youssef-ar merged commit 4c4b797 into main Apr 6, 2026
6 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.

1 participant