Skip to content

HOTFIX: Read final tag from env#122

Merged
prasadlohakpure merged 2 commits into
mainfrom
hf/tag_fix
Jul 15, 2026
Merged

HOTFIX: Read final tag from env#122
prasadlohakpure merged 2 commits into
mainfrom
hf/tag_fix

Conversation

@prasadlohakpure

Copy link
Copy Markdown
Collaborator

This pull request makes a minor update to the Docker image workflow by switching from using the output of the tagger step to the DOCKER_TAG environment variable for tagging and inspecting Docker images. This change helps standardize how the tag is referenced throughout the workflow.

  • Updated the Docker image tagging and inspection steps in .github/workflows/docker-image.yml to use the DOCKER_TAG environment variable instead of ${{ steps.tagger.outputs.tag }} for more consistent and maintainable workflow configuration.

Copilot AI review requested due to automatic review settings July 15, 2026 08:23

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the Docker image publishing workflow to reference the computed Docker tag via a single DOCKER_TAG value, aiming to standardize tag usage when creating and inspecting the multi-arch manifest.

Changes:

  • Switch manifest creation to use DOCKER_TAG instead of steps.tagger.outputs.tag.
  • Switch manifest inspection to use DOCKER_TAG instead of steps.tagger.outputs.tag.

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

Comment thread .github/workflows/docker-image.yml
Comment thread .github/workflows/docker-image.yml
@prasadlohakpure prasadlohakpure merged commit 5aba4a3 into main Jul 15, 2026
8 checks passed
@prasadlohakpure prasadlohakpure deleted the hf/tag_fix branch July 15, 2026 08:29
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.

3 participants