feat(autodiscovery): bazel support#7568
Conversation
…into feat/bazel-mod-support
|
Wow that was fast :D Thanks for the pull request, I am going to need some time to review it |
olblak
added
enhancement
There was a problem hiding this comment.
Pull request overview
This pull request implements comprehensive Bazel Mod (Bzlmod) support for Updatecli, adding the ability to manage and auto-update Bazel module dependencies defined in MODULE.bazel files.
Changes:
- Added bazelregistry source/condition plugin to query the Bazel Central Registry for available module versions
- Added bazelmod source/condition/target plugin to read and update MODULE.bazel files
- Added bazel autodiscovery plugin to automatically generate update manifests for Bazel module dependencies
Reviewed changes
Copilot reviewed 34 out of 34 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| pkg/plugins/resources/bazelregistry/*.go | Implements querying Bazel Central Registry with version filtering, yanked version handling, and HTTP timeout configuration |
| pkg/plugins/resources/bazelmod/*.go | Implements reading/updating MODULE.bazel files with custom parser supporting multi-line bazel_dep() calls and semantic version comparison |
| pkg/plugins/autodiscovery/bazel/*.go | Implements automatic discovery of MODULE.bazel files with matching rules for ignore/only patterns and manifest generation |
| pkg/core/pipeline/resource/main.go | Registers bazelmod and bazelregistry resource plugins |
| pkg/core/pipeline/autodiscovery/main.go | Registers bazel autodiscovery plugin |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Completed the code review |
…into feat/bazel-mod-support
Signed-off-by: Olblak <[email protected]>
|
I had another round and after some testing, I noticed that the Bazel autodiscovery wouldn't work due to the generated condition. I added a commit to remove them. |
|
Thanks, @josill, for the pull request. |
|
amazing ❤️ |
3 participants
Fix #6682
Summary
Implements Bazel Mod (Bzlmod) support for Updatecli with three new components:
Tests
cd pkg/plugins/resources/bazelmod && go test -v
cd pkg/plugins/resources/bazelregistry && go test -v
cd pkg/plugins/autodiscovery/bazel && go test -v
Additional Information
Checklist