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

Accept minor release version string as goversion #109

Merged
merged 1 commit into from
Jan 13, 2023

Conversation

cetteup
Copy link
Contributor

@cetteup cetteup commented Jan 11, 2023

The only way to use a minor release version right now is to pass the download URL for the release. Meaning you lose the benefit of setup-go.sh automatically adding the architecture it needs.

These changes would allow users to simply specify 1.19.5 instead of having to pass the URL - without changing any of the existing options.

@wangyoucao577
Copy link
Owner

Excellent! Would you please also add a test here to make sure it works well? https://github.com/cetteup/go-release-action/blob/49e097e3fbdce35f1cd04f70b403e891ccac745d/.github/workflows/autotest.yml#L82

@cetteup
Copy link
Contributor Author

cetteup commented Jan 11, 2023

Of course! I was looking for tests, but did not check the workflows. My bad.

I added a minor version to the setup-go-test now.

@wangyoucao577 wangyoucao577 merged commit fa1f667 into wangyoucao577:master Jan 13, 2023
@cetteup cetteup deleted the feat/go-minor-version branch January 13, 2023 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants