Skip to content

Update docker-publish.yml#31

Merged
dewitt4 merged 2 commits intomainfrom
lower-case
Oct 29, 2025
Merged

Update docker-publish.yml#31
dewitt4 merged 2 commits intomainfrom
lower-case

Conversation

@dewitt4
Copy link
Copy Markdown
Contributor

@dewitt4 dewitt4 commented Oct 29, 2025

No description provided.

@dewitt4 dewitt4 self-assigned this Oct 29, 2025
Copilot AI review requested due to automatic review settings October 29, 2025 18:31
@dewitt4 dewitt4 added the bug Something isn't working label Oct 29, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes Docker image naming issues by converting repository names to lowercase before using them in image references. GitHub Container Registry requires lowercase image names, and this change ensures compatibility by converting github.repository to lowercase.

  • Added a new step in both jobs to convert repository names to lowercase using shell commands
  • Updated all image references to use the lowercase outputs instead of environment variables
  • Applied changes consistently across both the API and Dashboard build jobs

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/docker-publish.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dewitt4 dewitt4 merged commit 8121a88 into main Oct 29, 2025
10 checks passed
@dewitt4 dewitt4 deleted the lower-case branch October 29, 2025 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants