Skip to content

Conversation

chappjc
Copy link
Member

@chappjc chappjc commented Mar 15, 2022

This updates for Go 1.18.

The supported Go versions are now 1.17 and 1.18.

CI is updated to new action versions and uses golangci-lint v1.45.0.

All the old //+build lines are removed now that all supported Go versions understand the new //go:build lines.

The go.mod now specifies go 1.17, and a go mod tidy is applied.

@chappjc chappjc force-pushed the go1.18 branch 2 times, most recently from 73ae9ce to 1359037 Compare March 18, 2022 15:59
@chappjc
Copy link
Member Author

chappjc commented Mar 18, 2022

Working now, but golangci-lint is still sorting out go 1.18 issues: golangci/golangci-lint#2649 (comment)

@chappjc chappjc marked this pull request as ready for review March 18, 2022 16:08
Copy link
Member

@JoeGruffins JoeGruffins left a comment

Choose a reason for hiding this comment

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

grep -rn "+build" will find three more usages in the dex/networks area

chappjc added 2 commits March 22, 2022 20:11
This updates the go.mod files to go 1.17, the new minimum supported
Go version.

Update README for go 1.18.

Remove all +build lines from go code since all supported Go versions
understand //go:build lines.
- Go versions 1.17 and 1.18
- set token permissions for build.yml
- update checkout and setup-go actions to v3
- golangci-lint 1.45.0
@chappjc chappjc merged commit d3c228c into decred:master Mar 23, 2022
@chappjc chappjc deleted the go1.18 branch March 23, 2022 19:35
@chappjc chappjc added this to the 0.5 milestone Apr 21, 2022
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.

2 participants