Skip to content

Fix buildifier for micromamba#220

Open
nettle wants to merge 1 commit intoEricsson:mainfrom
nettle:buildifier-patch
Open

Fix buildifier for micromamba#220
nettle wants to merge 1 commit intoEricsson:mainfrom
nettle:buildifier-patch

Conversation

@nettle
Copy link
Copy Markdown
Collaborator

@nettle nettle commented Apr 9, 2026

Why:
Buildifier, unfortunately does not work under Micromamba environment
because buildifier scans (find) all subdirectories including bazel output base dir
inside micromamba dir and some files cannot be read there, so it causes error.

What:

  • Add patches for buildifier to prune excluded locations
  • Test for Bazel 6.5 and 7.7

Test:

source .ci/micromamba/init.sh
bazel test :buildifier_native

Addresses: #155, #163

@nettle nettle requested review from Szelethus and furtib April 9, 2026 17:43
@nettle
Copy link
Copy Markdown
Collaborator Author

nettle commented Apr 9, 2026

Hi @furtib, @Szelethus,
Please see how patch technique is working in Bazel ;)
Yes, it is official!

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.

1 participant