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

maint: Add .tools-version file #188

Closed
wants to merge 1 commit into from
Closed

Conversation

MikeGoldsmith
Copy link
Contributor

@MikeGoldsmith MikeGoldsmith commented Mar 16, 2023

Which problem is this PR solving?

Adds a .tools-version file that indicates what version of go should be used with the project. In this case I matched what we build our binaries with in CircleCI (1.18).

Short description of the changes

  • add .tools-version file that sets go version to 1.18

@MikeGoldsmith MikeGoldsmith added the type: maintenance The necessary chores to keep the dust off. label Mar 16, 2023
@MikeGoldsmith MikeGoldsmith self-assigned this Mar 16, 2023
@MikeGoldsmith MikeGoldsmith requested a review from a team March 16, 2023 17:45
@MikeGoldsmith MikeGoldsmith changed the title [maint] Add .tools-version file maint: Add .tools-version file Mar 16, 2023
@@ -0,0 +1 @@
golang 1.18
Copy link
Member

Choose a reason for hiding this comment

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

Sadly, I think this will result in Go 1.18.0, not "the latest 1.18.x".

Copy link
Contributor Author

@MikeGoldsmith MikeGoldsmith Mar 20, 2023

Choose a reason for hiding this comment

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

Yeah, you're correct - it only enforces the major.minor and doesn't care about the patch. However, this is a good step forward to where we didn't enforce any version.

There's an open PR in asdf to try to tackle it but there's mixed opinion on whether it should be part of the core tool or the plugins.

Are you okay with this to start with, then if the asdf supports 'latest' patches we move to that?

@pkanal
Copy link
Contributor

pkanal commented Mar 21, 2023

Closing this PR based on @robbkidd's comment about asdf forcing .0 instead of .x

@pkanal pkanal closed this Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: maintenance The necessary chores to keep the dust off.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants