Update buildifier to 6.1.0

.bazelci/presubmit.yml

@@ -2,7 +2,7 @@
 validate_config: 1
 bazel: 6.0.0
 buildifier:
-  version: 5.1.0
+  version: 6.1.0
   # no lint warnings for the moment. They are basically a smoke alarm in hell.
   # keep this argument in sync with .pre-commit-config.yaml
   warnings: "-confusing-name,-constant-glob,-duplicated-name,-function-docstring,-function-docstring-args,-function-docstring-header,-module-docstring,-name-conventions,-no-effect,-constant-glob,-provider-params,-print,-rule-impl-return,-bzl-visibility,-unnamed-macro,-uninitialized,-unreachable"

.pre-commit-config.yaml

@@ -3,7 +3,7 @@
 # See https://pre-commit.com/hooks.html for more hooks
 repos:
   - repo: https://github.com/keith/pre-commit-buildifier
-    rev: 4.0.1.1
+    rev: 6.1.0
     hooks:
       - id: buildifier
         args: &args

kotlin/internal/js/js.bzl

@@ -36,7 +36,7 @@ _JS_STDLIB_MAP = {
 #       compiler, there is no 'module-name' flag available. So to keep things simple it's hard coded to the module name.
 def _lock_attrs(name, kwargs):
     if native.repository_name().startswith("@") and native.repository_name().endswith(_KT_COMPILER_REPO):
-        name = _JS_STDLIB_MAP.get(name, default = name)
+        name = _JS_STDLIB_MAP.get(name, name)
     if kwargs.get("module_root") != None:
         fail("The module_root is an internal attribute.")
     else:

src/test/kotlin/io/bazel/kotlin/builder/BUILD

@@ -50,10 +50,10 @@ java_library(
         "@com_github_jetbrains_kotlin//:kotlin-stdlib",
     ],
     deps = _COMMON_DEPS + [
+        "//src/main/kotlin/io/bazel/kotlin/builder/tasks",
         "//third_party:autovalue",
         "//third_party:dagger",
         "@kotlin_rules_maven//:javax_annotation_javax_annotation_api",
-        "//src/main/kotlin/io/bazel/kotlin/builder/tasks",
     ],
 )