Skip to content

fix: lowercase GHCR image name in release workflow#24

Merged
rophy merged 1 commit into
masterfrom
fix/release-lowercase
May 15, 2026
Merged

fix: lowercase GHCR image name in release workflow#24
rophy merged 1 commit into
masterfrom
fix/release-lowercase

Conversation

@rophy

@rophy rophy commented May 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Fix repository name must be lowercase error in release workflow
  • Lowercase github.repository before using as Docker image tag (same pattern as redo-log-tests.yaml)

Test plan

  • Merge, re-push v1.9.0.1 tag, verify release build succeeds

Summary by CodeRabbit

Release Notes

  • Chores
    • Optimized Docker image tagging process in the release workflow for improved consistency and reliability.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 15, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: c15e4721-4838-4f63-b869-5dd244a0adb9

📥 Commits

Reviewing files that changed from the base of the PR and between f17a7ce and 9241e3f.

📒 Files selected for processing (1)
  • .github/workflows/release.yaml

📝 Walkthrough

Walkthrough

The release workflow adds an environment variable normalization step. It defines an image name, converts it to lowercase via a dedicated step, and uses the normalized value in Docker build/push tags instead of inline expressions.

Changes

Docker Image Tag Normalization

Layer / File(s) Summary
Image tag normalization for Docker build/push
.github/workflows/release.yaml
Workflow defines env.IMAGE with the repository image path, adds a step to lowercase it and export to GITHUB_ENV, then updates the Docker build/push action to reference the normalized environment variable in tags.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • rophy/OpenLogReplicator#23: Both PRs modify .github/workflows/release.yaml to build/push the release Docker image to GHCR, with this PR specifically changing how the image tag is computed via a lowercased env.IMAGE variable on top of the release workflow introduced in #23.

Poem

A workflow so clean, with lowercase dreams,
Docker tags flow through env.IMAGE streams,
Normalize once, use it twice with grace,
Build and push image to the registry space! 🐇✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 change: lowercasing the GHCR image name in the release workflow to fix repository name validation errors.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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 fix/release-lowercase

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.

@rophy rophy merged commit 0c23e8c into master May 15, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant