Skip to content

gha: Log Go version in release summaries - #499

Merged
crazy-max merged 1 commit into
docker:mainfrom
vvoland:release-gover
Aug 19, 2026
Merged

gha: Log Go version in release summaries#499
crazy-max merged 1 commit into
docker:mainfrom
vvoland:release-gover

Conversation

@vvoland

@vvoland vvoland commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

Expose the Go version resolved by bake so release summaries record the actual toolchain used, including package defaults and workflow overrides.

Expose the Go version resolved by bake so release summaries record the
actual toolchain used, including package defaults and workflow
overrides.

Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@vvoland vvoland self-assigned this Aug 19, 2026
@vvoland
vvoland requested a review from a team as a code owner August 19, 2026 09:53

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

- build
- verify
env:
GO_VERSION: ${{ needs.prepare.outputs.go-version }}

@crazy-max crazy-max Aug 19, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hum actually would be better not name it GO_VERSION to avoid colliding with bake def var

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thinking again that seems fine for release target

@crazy-max
crazy-max merged commit e018247 into docker:main Aug 19, 2026
1414 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants