-
-
Notifications
You must be signed in to change notification settings - Fork 597
deps(gazelle): Bump rules_go from 0.41.0 to 0.55.0; gazelle from 0.31.0 (or 0.33.0 for bzlmod) to 0.40.0; go from 1.19.4 to 1.22.0 #2962
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
base: main
Are you sure you want to change the base?
Conversation
Bump `rules_go` from 0.41.0 to 0.55.0 Fixes bazel-contrib#2956. + Run `go get github.com/bazelbuild/[email protected]` for version bump. + Run `go mod tidy` to clean up stale deps + Run `bazel run //:gazelle_update_repos` + Figure out http_archive hashes and apply them for WORKSPACE users. + Also bump bazel-gazelle to address a warning: + `go get github.com/bazelbuild/[email protected]` + > Mismatch between versions requested for module > github.com/bazelbuild/bazel-gazelle > Bazel dependency version requested in MODULE.bazel: 0.40.0 > Go module version requested in go.mod: 0.31.1 > Please resolve this mismatch to prevent discrepancies between > native Go and Bazel builds + Update CHANGELOG.md accordingly
I'm not exactly sure how to fix the failing WORKSPACE tests.
My assumption is that there's an issue with this go_repository being a nested module, and that go_repository(
name = "org_golang_x_tools_go_vcs",
importpath = "golang.org/x/tools/go/vcs",
sum = "h1:cOIJqWBl99H1dH5LWizPa+0ImeeJq3t3cJjaeOWUAL4=",
version = "v0.1.0-deprecated",
) Any thoughts? I thought that maybe some Thus far I'm unable to reproduce the issue locally, so fixing it might result in some CI spam... |
Reproduced via the very simple Investigating a fix. |
@aignas I've solved the WORKSPACE stuff by bumping Example:
Can you take a look? |
…ORKSPACE + 0.31, 0.32 errored out with the golang_x_tools//go/vcs issue from before + 0.33, 0.34, 0.35 have the wrong load sorting.
According to bazel-contrib/bazel-gazelle#1654 we need to call the gazelle_dependencies macro in WORKSPACE. We already do that in gazelle/WORKSPACE but it's not in the root WORKSPACE. Maybe we have to do it there, too...
Huzzah! I got the rest of the Gazelle stuff passing for WORKSPACE. Now I need to investigate the Windows builds and my tree-sitter fork... |
Bump versions:
rules_go
from 0.41.0 to 0.55.0gazelle
from 0.31.0 (or 0.33.0 for bzlmod) to 0.40.0go
from 1.19.4 to 1.22.0Fixes #2956.
go get github.com/bazelbuild/[email protected]
for version bump.go mod tidy
to clean up stale depsbazel run //:gazelle_update_repos
go get github.com/bazelbuild/[email protected]