Skip to content

dev: fix GO_VERSION in post release workflow #4926

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 19, 2024
Merged

dev: fix GO_VERSION in post release workflow #4926

merged 1 commit into from
Aug 19, 2024

Conversation

silas
Copy link
Contributor

@silas silas commented Aug 19, 2024

There are multiple jobs in the GitHub post-release workflow that reference the GO_VERSION environment variable, but it is only defined for the update-docs (aka Update readme) job.

This means that the static assets for version 1.60.1 are built with Go 1.22 instead of 1.23.

You can confirm this by comparing the version of Go used in the actions/setup-go@5 step of the Update readme job to the one in Update GitHub Action assets.

Copy link

boring-cyborg bot commented Aug 19, 2024

Hey, thank you for opening your first Pull Request !

@CLAassistant
Copy link

CLAassistant commented Aug 19, 2024

CLA assistant check
All committers have signed the CLA.

@ldez ldez self-requested a review August 19, 2024 23:22
@ldez ldez changed the title Fix GO_VERSION in post release workflow docs: fix GO_VERSION in post release workflow Aug 19, 2024
@ldez ldez added bug Something isn't working area: docs area: ci PR that update CI labels Aug 19, 2024
@ldez ldez added this to the next milestone Aug 19, 2024
@ldez ldez changed the title docs: fix GO_VERSION in post release workflow dev: fix GO_VERSION in post release workflow Aug 19, 2024
@ldez
Copy link
Member

ldez commented Aug 19, 2024

This means that the static assets for version 1.60.1 are built with Go 1.22 instead of 1.23.

This only impacts the generators of docs and JSON files, there is no impact on the documentation or the generated public assets.

@ldez ldez merged commit 741df1f into golangci:master Aug 19, 2024
16 checks passed
@silas silas deleted the fix-post-release-go-version branch August 19, 2024 23:34
@ldez ldez modified the milestones: next, v1.60 Aug 20, 2024
uudashr pushed a commit to uudashr/golangci-lint that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ci PR that update CI area: docs bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants