Skip to content
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

Add a go.mod toolchain version #4347

Merged
merged 1 commit into from
May 16, 2024
Merged

Conversation

Jamstah
Copy link
Collaborator

@Jamstah Jamstah commented May 13, 2024

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
Member

Curious: what are we gaining with this?

@Jamstah
Copy link
Collaborator Author

Jamstah commented May 14, 2024

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.

Copy link
Member

@milosgajdos milosgajdos left a comment

Choose a reason for hiding this comment

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

@milosgajdos milosgajdos merged commit e0a54de into distribution:main May 16, 2024
16 checks passed
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.

None yet

3 participants