Skip to content

Update to Go 1.17.0#42777

Merged
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:update_go_1.17
Aug 26, 2021
Merged

Update to Go 1.17.0#42777
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:update_go_1.17

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Aug 23, 2021

Opening as draft to see if things explode

Depends on:

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@thaJeztah
Copy link
Member Author

🤦 Forgot to gofmt with Go 1.17, because it changes // +build to //go:build

[2021-08-23T13:29:02.082Z] cmd/dockerd/service_unsupported.go:1: File is not `goimports`-ed (goimports)
[2021-08-23T13:29:02.082Z] // +build !windows

@thaJeztah
Copy link
Member Author

Rebased, because #41479 was merged, which changed adjacent lines in Dockerfile.windows (rebase was otherwise clean)

@thaJeztah
Copy link
Member Author

@tianon @cpuguy83 @tonistiigi PTAL

Copy link
Member

@tianon tianon 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 to be explicit, I am against backporting this to 20.10)

@thaJeztah thaJeztah merged commit 2b70006 into moby:master Aug 26, 2021
@thaJeztah thaJeztah deleted the update_go_1.17 branch August 26, 2021 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants