chore: enable unnamed-macro buildifier check (#328)

bazelbuild · Oct 3, 2023 · 8e17eff · 8e17eff
1 parent 27d7a1f
commit 8e17eff
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -15,4 +15,4 @@ repos:
       - id: buildifier
       - id: buildifier-lint
         args:
-          - --warnings=-provider-params,-unnamed-macro
+          - --warnings=-provider-params
diff --git a/bzlmod/04-local_config_and_register_toolchains/local_config_sh.bzl b/bzlmod/04-local_config_and_register_toolchains/local_config_sh.bzl
@@ -39,10 +39,10 @@ my_sh_config = repository_rule(
 )
 
 # Used by WORKSPACE
-def sh_configure():
+def sh_configure(name = "my_local_config_sh"):
     """Detect the local shell interpreter and register its toolchain."""
-    my_sh_config(name = "my_local_config_sh")
-    native.register_toolchains("@my_local_config_sh//:local_sh_toolchain")
+    my_sh_config(name)
+    native.register_toolchains("@{}//:local_sh_toolchain".format(name))
 
 # Used by MODULE.bazel
 my_sh_config_extension = module_extension(