Skip to content

898: Re-added build image step to ci-cd#886

Merged
Arshadul-Monir merged 1 commit intomainfrom
898
Apr 4, 2026
Merged

898: Re-added build image step to ci-cd#886
Arshadul-Monir merged 1 commit intomainfrom
898

Conversation

@Arshadul-Monir
Copy link
Copy Markdown
Contributor

@Arshadul-Monir Arshadul-Monir commented Apr 4, 2026

898

Description of changes

Added back the previously removed buildImage step in the CI/CD workflow to push changes in main to prod.

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

Title

898: Re-added build image step to ci-cd


PR Type

Enhancement


Description

  • Re-adds buildImage job to CI/CD.

  • Triggers on main branch pushes.

  • Depends on schema validation and tests.


Diagram Walkthrough

flowchart LR
  validateDBSchema["Validate DB Schema"] --> buildImage
  backendTests["Run Backend Tests"] --> buildImage
  frontendTests["Run Frontend Tests"] --> buildImage
  buildImage["Build Docker Image & Upload"]
Loading

File Walkthrough

Relevant files
Configuration changes
ci-cd.yml
Re-enable Docker image build job in CI/CD workflow             

.github/workflows/ci-cd.yml

  • Re-added the buildImage job to the CI/CD workflow.
  • Configured buildImage to run only on pushes to the main branch.
  • Established dependencies on validateDBSchema, backendTests, and
    frontendTests jobs.
  • Utilized the /.github/composite/build-image action for the build
    process.
+17/-0   

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@Arshadul-Monir
Copy link
Copy Markdown
Contributor Author

/ai

Comment thread .github/workflows/ci-cd.yml
@tahminator
Copy link
Copy Markdown
Member

/review

@tahminator
Copy link
Copy Markdown
Member

/describe

@tahminator
Copy link
Copy Markdown
Member

/improve

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

Title

898: Re-added build image step to ci-cd


PR Type

Enhancement


Description

  • Re-added Docker image build job

  • Integrates into CI/CD workflow

  • Depends on schema and test validations

  • Executes only on main branch pushes


Diagram Walkthrough

flowchart LR
  validateDBSchema["Validate DB Schema"] --> buildImage["Build Docker Image & Upload to Registry"]
  backendTests["Run Backend Tests"] --> buildImage
  frontendTests["Run Frontend Tests"] --> buildImage
Loading

File Walkthrough

Relevant files
Configuration changes
ci-cd.yml
Re-added Docker image build and upload job to CI/CD.         

.github/workflows/ci-cd.yml

  • Re-added the buildImage job to the CI/CD workflow.
  • Configured buildImage to run on ubuntu-latest and depend on
    validateDBSchema, backendTests, and frontendTests.
  • Ensured the buildImage job only executes when changes are pushed to
    the main branch.
  • Utilized the actions/checkout@v6 and /.github/composite/build-image
    actions for the build process.
+17/-0   

Comment thread .github/workflows/ci-cd.yml
@Arshadul-Monir
Copy link
Copy Markdown
Contributor Author

/ai

@tahminator
Copy link
Copy Markdown
Member

/review

@tahminator
Copy link
Copy Markdown
Member

/describe

@tahminator
Copy link
Copy Markdown
Member

/improve

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

Title

898: Re-added build image step to ci-cd


PR Type

Enhancement


Description

  • Re-added buildImage job to CI/CD.

  • Triggers on main branch pushes.

  • Depends on schema, backend, frontend tests.

  • Utilizes build-image composite action.


Diagram Walkthrough

flowchart LR
  A[validateDBSchema] --> B(buildImage)
  C[backendTests] --> B
  D[frontendTests] --> B
Loading

File Walkthrough

Relevant files
Configuration changes
ci-cd.yml
Re-add Docker image build step to CI/CD workflow                 

.github/workflows/ci-cd.yml

  • Re-added the buildImage job to the CI/CD workflow.
  • Configured buildImage to run only on pushes to the main branch.
  • Set buildImage to depend on validateDBSchema, backendTests, and
    frontendTests jobs.
  • The job uses the /.github/composite/build-image action, passing GPG
    secrets.
+17/-0   

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 4, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Comment thread .github/workflows/ci-cd.yml
@Arshadul-Monir
Copy link
Copy Markdown
Contributor Author

/deploy

@Arshadul-Monir Arshadul-Monir merged commit 26d5d67 into main Apr 4, 2026
38 checks passed
@Arshadul-Monir Arshadul-Monir deleted the 898 branch April 4, 2026 03:20
@Arshadul-Monir Arshadul-Monir restored the 898 branch April 4, 2026 04:16
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