Add Bazel CI workflow and buildifier formatting#459
Open
DominikAFischer wants to merge 8 commits into
Open
Conversation
7c8a332 to
e90a3b7
Compare
10d3c88 to
179b9f8
Compare
- Add bazel github action workflow - Add buildifier_prebuilt 8.5.1.1 dev dependency to MODULE.bazel - Add //:format_check and //:format_fix targets via buildifier_prebuilt Change-Id: Ie1a9a60d88336ebce2785e9ccb80d30053c1175a
Change-Id: Ib2857ad50f21541c7dfc12013fbb05bf633f0fac
Change-Id: I971a88188dbbd94d8537ebbe8194e0dd6c84c29b
Change-Id: I16bff2f980fb8623a8399170cd2cf9ff0b62c94b
Change-Id: I86bb6733e924cce83d939023e4016ddc09eed089
Change-Id: Ie7a8c13a96a03f0576a79d151788fc3af0a69ef5
Change-Id: I2cae6ce5f25c93feff46696aabcfdfd808654a03
de479a7 to
e7f77d1
Compare
Change-Id: I432d900f7d86ca5b5a097fdc8d00c21b0ffe8581
518b5b1 to
cd58d6e
Compare
949eb8f to
897f37f
Compare
Contributor
Author
|
False positive / flaky test |
897f37f to
cd58d6e
Compare
christophruethingbmw
approved these changes
May 29, 2026
Contributor
christophruethingbmw
left a comment
There was a problem hiding this comment.
Looks good to me, just wait for the fix of the flaky test, PR for that should come soon.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ℹ️ NOTE: The bulk of the changes made by this PR is the automatically generated Bazel lockfile: +2615 / -1
Add Bazel CI workflow and buildifier formatting:
bazel versionbazel mod deps --lockfile_mode=errorbazel run //:format_checkbazel query //...bazel build //...bazel build --config=s32k148 //...bazel test //...NOTE: Docker bind mount permission issues:
Bazelisk/Bazel cache directories created by Docker bind mounts via
create_host_path: trueare owned by root, causing permission errors for the unprivileged container user. It seems this is a fundamental, unresolved Docker Engine limitation (moby/moby#2259 has been open since 2013).Instead
create_host_path: falseis used and it's the users responsibility to set up the cache dirs locally. This will be added to the README and Bazel quick start guide once the Bazel related sections are added.NOTE: We have decided to keep two instances of buildifier for now: