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

Use latest Go for releases #115

Merged
merged 2 commits into from
Apr 30, 2024
Merged

Use latest Go for releases #115

merged 2 commits into from
Apr 30, 2024

Conversation

nineinchnick
Copy link
Member

#96 bumped the minimum required version to 1.21, so running releases with 1.20 will likely fail.

Also run the CI workflow only on pushes to the master branch. Now when working on PRs from forks, it runs both in the target repo and in the fork, which is unnecessary.

@cla-bot cla-bot bot added the cla-signed label Apr 30, 2024
@wendigo
Copy link
Contributor

wendigo commented Apr 30, 2024

@nineinchnick ping me when CI is green

@wendigo wendigo merged commit d4afa69 into trinodb:master Apr 30, 2024
5 checks passed
@nineinchnick nineinchnick deleted the workflows branch April 30, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants