fix: update rules_android to 0.7.1 to support bazel 9 - #6
Conversation
Upgraded rules_android to version 0.7.1 in MODULE.bazel to resolve build failures occurring due to the removal of the CcInfo symbol in Bazel 9.0. Also updated Gazelle's generated build targets. Co-authored-by: filmil <246576+filmil@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
| data = ["names.txt"], | ||
| importpath = "github.com/filmil/synod/internal/names", | ||
| visibility = ["//visibility:public"], | ||
| data = ["names.txt"], |
There was a problem hiding this comment.
I have restored this data attribute by reverting the modifications made to this file by Gazelle.
| data = ["//third_party/bootstrap"], | ||
| importpath = "github.com/filmil/synod/internal/server", | ||
| visibility = ["//visibility:public"], | ||
| data = ["//third_party/bootstrap"], |
There was a problem hiding this comment.
I have restored this data attribute by reverting the modifications made to this file by Gazelle.
The previous commit mistakenly included changes to `BUILD.bazel` files generated by `bazel run //:gazelle`. This commit reverts those changes as requested in the PR review, keeping only the update to `rules_android` in `MODULE.bazel`. Co-authored-by: filmil <246576+filmil@users.noreply.github.com>
1 participant
Fixes a build failure caused by upgrading to Bazel 9, which removed the
CcInfosymbol fromrules_androidversion0.6.4and older. Upgrading to0.7.1fixes the issue. Also ranbazel run //:gazelleandbazel run @rules_go//go -- mod tidyto clean up theBUILD.bazelfiles.PR created automatically by Jules for task 11660525259255278655 started by @filmil