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

update CI go version. #1210

Merged
merged 1 commit into from
Jan 20, 2023
Merged

update CI go version. #1210

merged 1 commit into from
Jan 20, 2023

Conversation

mindaugasbarcauskas
Copy link
Contributor

What changed?
Updating docker files to use 1.19 go version in CI pipelines

Why?

How did you test it?

CI pipeline passes.

Potential risks

Copy link
Contributor

@davidporter-id-au davidporter-id-au left a comment

Choose a reason for hiding this comment

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

LGTM, but maybe run it by @Groxx , I have less strong opinions here

@Groxx
Copy link
Contributor

Groxx commented Jan 19, 2023

yea, I think we can do this. it'll let us start formatting for 1.19 and then we can get the godoc improvements too.

I believe the go.mod value will still prevent us from using generics and stuff that doesn't exist in 1.13 (which is good). we should probably consider bumping that to at least 1.18 so we can use generics at some point. 1.18 is about 9 months old now, and 1.13 is ancient, seems probably safe to update and force that on our users.

@Groxx
Copy link
Contributor

Groxx commented Jan 19, 2023

will we want to include 1.19-fmt in this commit, or a later one, or did the earlier one take care of that? I forget how all this works with our CI pipeline, but it's worth keeping the inconsistent-fmt gap small.

@mindaugasbarcauskas mindaugasbarcauskas merged commit 186f416 into master Jan 20, 2023
mindaugasbarcauskas added a commit that referenced this pull request Jan 20, 2023
mindaugasbarcauskas added a commit that referenced this pull request Jan 20, 2023
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.

3 participants