pin buildifier (#1131)

bazelbuild · Feb 9, 2022 · c9e5b63 · c9e5b63
1 parent f569827
commit c9e5b63
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/.bazelci/presubmit.yml b/.bazelci/presubmit.yml
@@ -302,5 +302,9 @@ tasks:
     # TODO: fix test targets in `crate_universe_windows_targets`
     test_targets: *crate_universe_windows_targets
 buildifier:
-  version: latest
+  # TODO: Version 5.0.0 introduced a ton of unused variable warnings in cases
+  # that should have been accounted for by `_` prefixes on the variable. Buildifier
+  # should ignore `_` prefixed variables when checking if something is unused
+  # https://github.com/bazelbuild/buildtools/issues/1044
+  version: 4.2.5
   warnings: "all"