-
Notifications
You must be signed in to change notification settings - Fork 14
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
Update to Go 1.22 and update deps #78
Conversation
go mod tidy
0b6792a
to
15b8f06
Compare
b083627
to
d80e1cb
Compare
GOIMPORTS := go run golang.org/x/tools/cmd/goimports@latest | ||
GOLICENSER := go run github.com/elastic/go-licenser@latest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggest that if @latest
are used, the makefile emits the version of the tools during the build.
curl -s https://proxy.golang.org/golang.org/x/tools/@latest | jq '.Version'
curl -s https://proxy.golang.org/github.com/elastic/go-licenser/@latest | jq '.Version'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think what you are saying is that we should not use @latest
? Based on recent pains, I would agree. Moving toward a more hermetic build would be better despite the increased effort to periodically update versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer not to have dynamic versions, yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I opened #80 to automate this.
Update to Go 1.22.
Update dependencies.
Bind to 127.0.0.1 when testing.
Add dependabot.yml config to keep GH actions updated.
Remove usage of
GO111MODULE=off
.Update to https://github.com/updatecli/updatecli-action/releases/tag/v2.56.0.