Skip to content

Add a go.mod toolchain version#4347

Merged
milosgajdos merged 1 commit into
distribution:mainfrom
Jamstah:go-toolchain
May 16, 2024
Merged

Add a go.mod toolchain version#4347
milosgajdos merged 1 commit into
distribution:mainfrom
Jamstah:go-toolchain

Conversation

@Jamstah

@Jamstah Jamstah commented May 13, 2024

Copy link
Copy Markdown
Collaborator

go 1.21 added toolchain support. We should now specify a toolchain version in go.mod.

https://go.dev/doc/toolchain

go 1.21 added toolchain support. We should now specify a toolchain
version in go.mod.

https://go.dev/doc/toolchain

Signed-off-by: James Hewitt <james.hewitt@uk.ibm.com>
@github-actions github-actions Bot added the dependencies Pull requests that update a dependency file label May 13, 2024
@milosgajdos

Copy link
Copy Markdown
Member

Curious: what are we gaining with this?

@Jamstah

Jamstah commented May 14, 2024

Copy link
Copy Markdown
Collaborator Author

For one, the removal of a security warning: https://github.com/distribution/distribution/security/code-scanning/tools/CodeQL/status/configurations/actions-FZTWS5DIOVRC653POJVWM3DPO5ZS6Y3PMRSXC3BNMFXGC3DZONUXGLTZNVWA/aeb963eb3dd5cba62a5b17e8ad9ab76aeb67a2c8d843237c097778647c7eaf69

Other than that, assurance that developers are using a toolchain that is as new as the one we are building the project with.

@milosgajdos milosgajdos 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.

@milosgajdos milosgajdos requested review from squizzi and thaJeztah May 14, 2024 11:47
@milosgajdos milosgajdos merged commit e0a54de into distribution:main May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants