Skip to content
This repository has been archived by the owner on Mar 19, 2024. It is now read-only.

update to Go v1.19 #424

Merged
merged 5 commits into from
Oct 28, 2022
Merged

update to Go v1.19 #424

merged 5 commits into from
Oct 28, 2022

Conversation

mikemorris
Copy link
Contributor

@mikemorris mikemorris commented Oct 21, 2022

Changes proposed in this PR:

  • Update to Go v1.19
  • Update links in release metadata
  • Add Makefile tools to internal/tools.go following existing pattern, to allow versioning through go.mod
  • Update Consul patch releases in CI tests

How I've tested this PR:

Running all CI tests including conformance.

How I expect reviewers to test this PR:

Verify that updates seem reasonable.

Checklist:

  • Tests added
  • CHANGELOG entry added

    Run make changelog-entry for guidance in authoring a changelog entry, and
    commit the resulting file, which should have a name matching your PR number.
    Entries should use imperative present tense (e.g. Add support for...)

@mikemorris mikemorris added the pr/conformance Run conformance tests from kubernetes-sigs/gateway-api label Oct 21, 2022
@mikemorris mikemorris marked this pull request as ready for review October 21, 2022 18:38
@mikemorris mikemorris requested review from a team, samsalisbury and mdeggies October 21, 2022 18:38
@nathancoleman
Copy link
Member

@mikemorris can you update the pin in go.mod? Otherwise LGTM. Exciting update!

@mikemorris
Copy link
Contributor Author

can you update the pin in go.mod

I was under the impression that should only updated when actually necessary (e.g. introducing usage of a new feature like generics) but I suppose it could be easy to miss that instance if we're only developing/building/testing on Go 1.19 😐

Thoughts?

@nathancoleman
Copy link
Member

that makes sense

@nathancoleman
Copy link
Member

@mdeggies @samsalisbury could one of you please review when you get a moment on behalf of RelEng?

@nathancoleman nathancoleman merged commit 00e038f into main Oct 28, 2022
@nathancoleman nathancoleman deleted the go/1.19 branch October 28, 2022 15:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr/conformance Run conformance tests from kubernetes-sigs/gateway-api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants