buildifier-prebuilt
buildifier-prebuilt copied to clipboard
keith
A bazel toolchain for using prebuilt binaries for buildifier and buildozer
The current [README.md](/README.md) states that the following should work: ```sh bazel run -- @buildifier_prebuilt//:buildozer ARGS ``` However, this will not work correctly. Bazel will execute the command in a sandbox,...
The only reason this repo uses toolchains is to avoid downloading multiple versions of buildifier that you'll never use on your platform. It looks like if you use some indirection...
The rule here has diverged a bit from [buildtools](https://github.com/bazelbuild/buildtools/blob/master/buildifier/internal/factory.bzl), namely that buildtools wraps the "diff" mode in a test rule useful for CI checks https://github.com/bazelbuild/buildtools/blob/master/buildifier/buildifier.bzl#L37-L41 I was going to adopt...
When I read "current time" I didn't know whether it meant: * release time * bazel sync time * command runtime Digging in, the version is hardcoded: https://github.com/keith/buildifier-prebuilt/blob/fa9175bbbcbfb5a782d262af73c2cb36b9cab99b/defs.bzl#L39-L41 https://github.com/keith/buildifier-prebuilt/blob/0b8177deda07ab95f69539c5536623b0678a169a/buildtools.bzl#L146-L161 I...
**Steps to reproduce** ``` $ echo 7.0.0 > .bazelversion $ echo 'bazel_dep(name = "buildifier_prebuilt", version = "6.4.0", dev_dependency = True)' > MODULE.bazel $ bazelisk query '@buildifier_prebuilt//...' ``` **Expected behavior** List...
Resolve https://github.com/keith/buildifier-prebuilt/issues/86
This PR includes fixes to support check, fix and test on Windows. Fixes https://github.com/keith/buildifier-prebuilt/issues/88 Tested with bazel 7.2.1 and both --noenable_runfiles and --enable_runfiles All validations performed with --windows_enable_symlinks enabled for...
This PR adds unit tests for buildifier that fail to demonstrate the current issues with Windows. Because running fix modifies the source tree, I set them up as bazel shell...