Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

Upgrade to Go 1.17.2 #402

Merged
merged 3 commits into from
Oct 20, 2021

Conversation

sebrandon1
Copy link
Member

@sebrandon1 sebrandon1 commented Oct 20, 2021

Because of the ^ character, we were already building Go 1.17 anyways (check out latest GitHub build logs).

@hamadise and I discussed yesterday that we might as well be more specific in our versioning scheme and specifically point to 1.17.x because we were already building it.

Release notes: https://golang.org/doc/go1.17

@@ -22,3 +22,21 @@ require (
google.golang.org/grpc v1.41.0
gopkg.in/yaml.v2 v2.4.0
)

require (
Copy link
Member Author

Choose a reason for hiding this comment

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

See section in release changelog about Pruned module graphs for more information about this section.

@sebrandon1
Copy link
Member Author

sebrandon1 commented Oct 20, 2021

Note: The rebuild mocks check is going to fail until the changes in #395 merge.

EDIT: I copied in the necessary changes for the GitHub checks to start working.

@jc-rh jc-rh merged commit 8d1afa9 into redhat-best-practices-for-k8s:main Oct 20, 2021
@sebrandon1 sebrandon1 deleted the go_117_update branch October 20, 2021 19:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants