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

Add //go:build lines #285

Merged
merged 1 commit into from Jan 4, 2022
Merged

Add //go:build lines #285

merged 1 commit into from Jan 4, 2022

Conversation

tklauser
Copy link
Contributor

@tklauser tklauser commented Jan 4, 2022

Starting with Go 1.17, //go:build lines are preferred over // +build
lines, see https://golang.org/doc/go1.17#build-lines and
https://golang.org/design/draft-gobuild for details.

This change was generated by running Go 1.17 go fmt ./... which
automatically adds //go:build lines based on the existing // +build
lines.

Also update the corresponding GitHub action to use Go 1.17 gofmt.

Starting with Go 1.17, //go:build lines are preferred over // +build
lines, see https://golang.org/doc/go1.17#build-lines and
https://golang.org/design/draft-gobuild for details.

This change was generated by running Go 1.17 go fmt ./... which
automatically adds //go:build lines based on the existing // +build
lines.

Also update the corresponding GitHub action to use Go 1.17 gofmt.
@dsnet
Copy link
Collaborator

dsnet commented Jan 4, 2022

It's going to be a looong time until we remove the old style of build tags 😞

dsnet
dsnet approved these changes Jan 4, 2022
@dsnet dsnet merged commit 039e37c into google:master Jan 4, 2022
15 checks passed
@tklauser tklauser deleted the go-build-tags branch January 4, 2022 17:52
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.

None yet

2 participants