Skip to content

Review: release workflow lowercase Docker image names#7

Merged
Adhish-Krishna merged 1 commit intodevelopmentfrom
copilot/sub-pr-6
Feb 27, 2026
Merged

Review: release workflow lowercase Docker image names#7
Adhish-Krishna merged 1 commit intodevelopmentfrom
copilot/sub-pr-6

Conversation

Copy link
Contributor

Copilot AI commented Feb 27, 2026

github.repository_owner can contain uppercase letters (e.g., SimpleNotificationSystem), producing invalid ghcr.io image names in release notes since Docker registry names must be lowercase.

Changes

  • release.yml: Derive OWNER_LOWER via tr '[:upper:]' '[:lower:]' and use it to construct CORE_IMAGE/DASHBOARD_IMAGE shell variables, replacing inline ${{ env.CORE_IMAGE_NAME }}/${{ env.DASHBOARD_IMAGE_NAME }} expressions in the release notes Docker section
OWNER_LOWER=$(echo "${{ github.repository_owner }}" | tr '[:upper:]' '[:lower:]')
CORE_IMAGE="${{ env.REGISTRY }}/${OWNER_LOWER}/simplens-core"
DASHBOARD_IMAGE="${{ env.REGISTRY }}/${OWNER_LOWER}/simplens-dashboard"

build-and-push.yml is unaffected — docker/metadata-action@v5 handles lowercase conversion automatically during image push.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

Copilot AI changed the title [WIP] Update release workflow to keep names in lower case Review: release workflow lowercase Docker image names Feb 27, 2026
@Adhish-Krishna Adhish-Krishna merged commit 920a8e2 into development Feb 27, 2026
1 check 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.

2 participants