-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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
Chore: Upgrade Go to 1.21.3 #77304
Chore: Upgrade Go to 1.21.3 #77304
Conversation
@@ -1,6 +1,3 @@ | |||
//go:build go1.3 | |||
// +build go1.3 |
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.
with these tags, the build results are:
go build -ldflags -w -X main.version=10.3.0-pre -X main.commit=caa185df53 -X main.buildstamp=1698422981 -X main.buildBranch=use-golant-121 -o ./bin/grafana ./pkg/cmd/grafana
# github.com/grafana/grafana/pkg/web
pkg/web/macaron.go:48:14: predeclared any requires go1.18 or later (-lang was set to go1.21; check go.mod)
pkg/web/macaron.go:73:13: cannot use http.NotFound (value of type func(w http.ResponseWriter, r *http.Request)) as Handler value in argument to m.NotFound
pkg/web/macaron.go:107:6: type aliases requires go1.9 or later (-lang was set to go1.21; check go.mod)
pkg/web/macaron.go:122:15: invalid operation: handler (variable of type Handler) is not an interface
exit status 1
exit status 1
make[1]: *** [build-go] Error 1
@@ -635,7 +635,6 @@ embed.go @grafana/grafana-as-code | |||
/.github/workflows/issue-opened.yml @grafana/grafana-community-support | |||
/.github/workflows/metrics-collector.yml @torkelo | |||
/.github/workflows/milestone.yml @marefr | |||
/.github/workflows/ox-code-coverage.yml @grafana/explore-squad |
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.
removed since not used by explore anymore... see grafana/code-coverage#29 (comment)
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.
The @grafana/observability-logs was using that workflow to report coverage changes btw.
What is this feature?
Update the min go version to 1.21. (see also #76359)
Why do we need this feature?
This lets us use new features added in 1.21. This lets us use the new standard library functions rather than the ones in `"golang.org/x/exp/..." (eg slices+maps)
Who is this feature for?
Developers