Skip to content

v2: add v2/cmd/gazelle#2277

Merged
jayconrod merged 11 commits intomasterfrom
v2-3-binary
Jan 28, 2026
Merged

v2: add v2/cmd/gazelle#2277
jayconrod merged 11 commits intomasterfrom
v2-3-binary

Conversation

@jayconrod
Copy link
Copy Markdown
Contributor

What type of PR is this?

Other

What package or component does this PR mostly affect?

v2/cmd/gazelle

What does this PR do? Why is it needed?

The new entry point for v2. It just wraps update.Run. Doesn't need to do much more than that.

Which issues(s) does this PR fix?

For #2272

Other notes for review

Still a pretty old version, but a year newer than what we had before.
This gives us fancy range loops.

Removed some apparently unused compatibility definitions referencing
multiple versions of Go.

For #2272
go.work is unrelated to the Bazel build. It keeps my editor happy without
gopackagesdriver. Bazel needs to rebuild all of gazelle after every change
via go_repository_tools, so gopackagesdriver isn't really usable.

For #2272
This new package now implements Gazelle's core fix/update functionality.
cmd/gazelle and v2/cmd/gazelle will both we wrappers around this, though
the update-repos command will remain in v1, since we're dropping it in v2.

Also modifed go_repository_tools to work in module mode, since it would
actually require deeper changes to continue working in GOPATH mode.

For #2272
The new entry point for v2. It just wraps update.Run. Doesn't need to do
much more than that.

For #2272
@jayconrod jayconrod requested a review from fmeum January 26, 2026 00:33
Comment thread v2/cmd/gazelle/main.go
Base automatically changed from v2-2-extract-update to master January 28, 2026 21:41
@jayconrod jayconrod enabled auto-merge (squash) January 28, 2026 22:08
@jayconrod jayconrod merged commit b5b771c into master Jan 28, 2026
17 checks passed
@jayconrod jayconrod deleted the v2-3-binary branch January 28, 2026 22:13
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