Skip to content

fix: correct containerRegistry value in CI/CD pipeline configuration - #81

Merged
omarghatasheh merged 1 commit into
mainfrom
musa-cicd-patch
Mar 31, 2026
Merged

fix: correct containerRegistry value in CI/CD pipeline configuration#81
omarghatasheh merged 1 commit into
mainfrom
musa-cicd-patch

Conversation

@MusaMisto

@MusaMisto MusaMisto commented Mar 31, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Chores
    • Updated CI/CD deployment workflow to support manual triggering in addition to automatic deployments.

@coderabbitai

coderabbitai Bot commented Mar 31, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The deploy-main job trigger condition in the CI/CD workflow was updated to run on both push events to the main branch and manual workflow_dispatch triggers, expanding deployment initiation options while keeping all other workflow logic intact.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/ui-cicd.yml
Broadened deploy-main job execution condition to include manual workflow dispatch triggers alongside existing push-to-main automation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • omarghatasheh

Poem

🐰 A button to deploy, now shiny and new,
Manual triggers await, what's old becomes true,
Push or dispatch, either way will do,
The workflow hops forward in skies bright and blue! 🚀

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title mentions 'containerRegistry value' but the actual change broadens the deploy-main job trigger condition to include workflow_dispatch, not containerRegistry configuration. Update the title to reflect the actual change, such as 'Add workflow_dispatch trigger to deploy-main job' or 'Enable manual triggering of CI/CD deployment workflow'.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch musa-cicd-patch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@omarghatasheh
omarghatasheh merged commit fcdca6f into main Mar 31, 2026
2 of 3 checks passed
@MusaMisto
MusaMisto deleted the musa-cicd-patch branch June 17, 2026 11:21
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