Skip to content

fix(workflow): update Docker metadata tags to use 'tag' event instead of 'branch' and 'pr'#2492

Merged
jihun merged 2 commits into
mainfrom
fix/docker-build-workflow
Jan 20, 2026
Merged

fix(workflow): update Docker metadata tags to use 'tag' event instead of 'branch' and 'pr'#2492
jihun merged 2 commits into
mainfrom
fix/docker-build-workflow

Conversation

@chiol
Copy link
Copy Markdown
Contributor

@chiol chiol commented Jan 20, 2026

Summary by CodeRabbit

  • Chores
    • Bumped CLI package version to 1.0.11.
    • Updated Docker image build tag generation configurations for development and production environments.

✏️ Tip: You can customize this high-level summary in your review settings.

@chiol chiol requested a review from jihun January 20, 2026 04:22
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jan 20, 2026

Walkthrough

The PR updates Docker workflow configurations to modify metadata tag generation triggers for both dev and prod environments, removing the semver major.minor pattern. Additionally, the CLI package version is incremented from 1.0.10 to 1.0.11.

Changes

Cohort / File(s) Summary
Docker Metadata Configuration
.github/workflows/docker-dev-image.yml, .github/workflows/docker-prod-image.yml
Removes semver major.minor pattern (type=semver,pattern={{major}}.{{minor}}) from metadata configurations for both API and Web services. Dev workflow uses tag event triggers; prod workflow changes from pull request to tag event trigger.
Package Version
apps/cli/package.json
Updates CLI package version from 1.0.10 to 1.0.11.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main changes in the pull request, which involve updating Docker metadata configurations in workflow files to use 'tag' event triggers instead of 'branch' and 'pr' events.
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.


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.

@jihun jihun merged commit a39995e into main Jan 20, 2026
8 checks passed
@jihun jihun deleted the fix/docker-build-workflow branch January 20, 2026 05: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