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

Golang 1.21.12 #357

Merged
merged 5 commits into from
Jul 5, 2024
Merged

Golang 1.21.12 #357

merged 5 commits into from
Jul 5, 2024

Conversation

michaelkaplan13
Copy link
Collaborator

Why this should be merged

Bump to latest 1.21 Golang release. Removes unnecessary explicit version used by actions/setup-go in favor of pulling directly from go.mod.

@@ -19,10 +19,6 @@ jobs:
path: awm-relayer
submodules: recursive

- name: Set Go version
run: |
source ./awm-relayer/scripts/versions.sh
Copy link
Collaborator

Choose a reason for hiding this comment

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

We still need to set GO_VERSION to provide as an arg to the Dockerfile build.

cam-schultz
cam-schultz previously approved these changes Jul 5, 2024
Copy link
Collaborator

@cam-schultz cam-schultz left a comment

Choose a reason for hiding this comment

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

LGTM. We should do a test run of the release job to make sure it still works. We can do so by pushing an rc tag

@michaelkaplan13 michaelkaplan13 merged commit 2b68ff2 into main Jul 5, 2024
7 checks passed
@michaelkaplan13 michaelkaplan13 deleted the golang-1.21.12 branch July 5, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants