Update buildifier hook and apply its fixes

.bcr/presubmit.yml

@@ -7,7 +7,7 @@ bcr_test_module:
       - ubuntu2004
       - macos
       - windows
-    bazel: [6.x, 7.x]
+    bazel: [6.x, 7.x, 8.x]
   tasks:
     run_test_module:
       name: Run test module

.pre-commit-config.yaml

@@ -14,13 +14,15 @@ default_language_version:
 repos:
   # Check formatting and lint for starlark code
   - repo: https://github.com/keith/pre-commit-buildifier
-    rev: 4.0.1.1
+    rev: 8.0.1
     hooks:
       - id: buildifier
+        args: [--warnings=-native-proto-info]
       - id: buildifier-lint
+        args: [--warnings=-native-proto-info]
   # Enforce that commit messages allow for later changelog generation
   - repo: https://github.com/commitizen-tools/commitizen
-    rev: v2.18.0
+    rev: v4.1.0
     hooks:
       # Requires that commitizen is already installed
       - id: commitizen

features.bzl

@@ -1,7 +1,7 @@
 """Defines all the features this module supports detecting."""
 
 load("@bazel_features_globals//:globals.bzl", "globals")
-load("//private:util.bzl", "ge", "ge_same_major", "gt", "lt")
+load("//private:util.bzl", "ge", "ge_same_major", "lt")
 
 _cc = struct(
     # Whether @bazel_tools//tools/cpp:optional_current_cc_toolchain and the `mandatory` parameter

private/globals_repo.bzl

@@ -34,6 +34,7 @@ bzl_library(
             fail("Invalid global name: %s" % global_)
 
         value = global_ if bazel_version < parse_version(version) else "None"
+
         # If the legacy_globals is available, we take the value from it.
         # The value is populated by --incompatible_autoload_externally and may apply to older Bazel versions
         lines.append("    %s = getattr(getattr(native, 'legacy_globals', None), '%s', %s)," % (global_, global_, value))
@@ -53,7 +54,7 @@ globals_repo = repository_rule(
         "legacy_globals": attr.string_dict(
             mandatory = True,
         ),
-     },
+    },
 )
 
 def _is_valid_identifier(s):

private/repos.bzl

@@ -1,14 +1,14 @@
 """Contains the macro bazel_features_repos to install internal repositories."""
 
+load("@bazel_tools//tools/build_defs/repo:utils.bzl", "maybe")
 load(":globals.bzl", "GLOBALS", "LEGACY_GLOBALS")
 load(":globals_repo.bzl", "globals_repo")
 load(":version_repo.bzl", "version_repo")
-load("@bazel_tools//tools/build_defs/repo:utils.bzl", "maybe")
 
 def bazel_features_repos():
     maybe(
         version_repo,
-        name = "bazel_features_version"
+        name = "bazel_features_version",
     )
     maybe(
         globals_repo,

private/util.bzl

@@ -19,6 +19,7 @@ def ge(v):
 
 def ge_same_major(v):
     pv = parse_version(v)
+
     # Version 1.2.3 parses to ([1, 2, 3], ...).
     return BAZEL_VERSION >= pv and BAZEL_VERSION[0][0] == pv[0][0]
 

test/test.bzl

@@ -1,8 +1,8 @@
 """Provides a macro to do some loading-time test assertions."""
 
-load("//private:util.bzl", "BAZEL_VERSION", "ge", "lt")
-load("//private:parse.bzl", "parse_version")
 load("//:features.bzl", "bazel_features")
+load("//private:parse.bzl", "parse_version")
+load("//private:util.bzl", "BAZEL_VERSION", "ge", "lt")
 
 def _empty_test_impl(ctx):
     extension = ".bat" if ctx.attr.is_windows else ".sh"