style: format .bzl files with buildifier (angular#23544)

PR Close angular#23544

Author: alexeagle

index.bzl

@@ -12,11 +12,13 @@ from source downstream. Alternately, this API is available from the
 used in a downstream project.
 """
 
-load("//packages/bazel:index.bzl",
+load(
+    "//packages/bazel:index.bzl",
     _ng_module = "ng_module",
     _ng_package = "ng_package",
     _protractor_web_test = "protractor_web_test",
-    _protractor_web_test_suite = "protractor_web_test_suite")
+    _protractor_web_test_suite = "protractor_web_test_suite",
+)
 load("//tools:ng_setup_workspace.bzl", _ng_setup_workspace = "ng_setup_workspace")
 
 ng_module = _ng_module

packages/bazel/index.bzl

@@ -10,9 +10,11 @@ Users should not load files under "/src"
 load("//packages/bazel/src:ng_module.bzl", _ng_module = "ng_module")
 load("//packages/bazel/src:ng_setup_workspace.bzl", _ng_setup_workspace = "ng_setup_workspace")
 load("//packages/bazel/src/ng_package:ng_package.bzl", _ng_package = "ng_package")
-load("//packages/bazel/src/protractor:protractor_web_test.bzl",
-     _protractor_web_test = "protractor_web_test",
-     _protractor_web_test_suite = "protractor_web_test_suite")
+load(
+    "//packages/bazel/src/protractor:protractor_web_test.bzl",
+    _protractor_web_test = "protractor_web_test",
+    _protractor_web_test_suite = "protractor_web_test_suite",
+)
 
 ng_module = _ng_module
 ng_package = _ng_package

packages/bazel/src/esm5.bzl

@@ -28,88 +28,92 @@ ESM5Info = provider(
 )
 
 def _map_closure_path(file):
-  result = file.short_path[:-len(".closure.js")]
-  # short_path is meant to be used when accessing runfiles in a binary, where
-  # the CWD is inside the current repo. Therefore files in external repo have a
-  # short_path of ../external/wkspc/path/to/package
-  # We want to strip the first two segments from such paths.
-  if (result.startswith("../")):
-    result = "/".join(result.split("/")[2:])
-  return result + ".js"
+    result = file.short_path[:-len(".closure.js")]
+
+    # short_path is meant to be used when accessing runfiles in a binary, where
+    # the CWD is inside the current repo. Therefore files in external repo have a
+    # short_path of ../external/wkspc/path/to/package
+    # We want to strip the first two segments from such paths.
+    if (result.startswith("../")):
+        result = "/".join(result.split("/")[2:])
+    return result + ".js"
 
 def _join(array):
-  return "/".join([p for p in array if p])
+    return "/".join([p for p in array if p])
 
 def _esm5_outputs_aspect(target, ctx):
-  if not hasattr(target, "typescript"):
-    return []
-
-  # We create a new tsconfig.json file that will have our compilation settings
-  tsconfig = ctx.actions.declare_file("%s_esm5.tsconfig.json" % target.label.name)
-
-  workspace = target.label.workspace_root if target.label.workspace_root else ""
-
-  # re-root the outputs under a ".esm5" directory so the path don't collide
-  out_dir = ctx.label.name + ".esm5"
-  if workspace:
-    out_dir = out_dir + "/" + workspace
-
-  outputs = [ctx.actions.declare_file(_join([out_dir, _map_closure_path(f)]))
-             for f in target.typescript.replay_params.outputs
-             if not f.short_path.endswith(".externs.js")]
-
-  ctx.actions.run(
-      executable = ctx.executable._modify_tsconfig,
-      inputs = [target.typescript.replay_params.tsconfig],
-      outputs = [tsconfig],
-      arguments = [
-          target.typescript.replay_params.tsconfig.path,
-          tsconfig.path,
-          _join([workspace, target.label.package, ctx.label.name + ".esm5"]),
-          ctx.bin_dir.path
-      ],
-  )
-
-  ctx.actions.run(
-      progress_message = "Compiling TypeScript (ES5 with ES Modules) %s" % target.label,
-      inputs = target.typescript.replay_params.inputs + [tsconfig],
-      outputs = outputs,
-      arguments = [tsconfig.path],
-      executable = target.typescript.replay_params.compiler,
-      execution_requirements = {
-          # TODO(alexeagle): enable worker mode for these compilations
-          "supports-workers": "0",
-      },
-  )
-
-  root_dir = _join([
-      ctx.bin_dir.path,
-      workspace,
-      target.label.package,
-      ctx.label.name + ".esm5",
-  ])
-
-  transitive_output={root_dir: depset(outputs)}
-  for dep in ctx.rule.attr.deps:
-    if ESM5Info in dep:
-      transitive_output.update(dep[ESM5Info].transitive_output)
-
-  return [ESM5Info(
-    transitive_output = transitive_output,
-  )]
+    if not hasattr(target, "typescript"):
+        return []
+
+    # We create a new tsconfig.json file that will have our compilation settings
+    tsconfig = ctx.actions.declare_file("%s_esm5.tsconfig.json" % target.label.name)
+
+    workspace = target.label.workspace_root if target.label.workspace_root else ""
+
+    # re-root the outputs under a ".esm5" directory so the path don't collide
+    out_dir = ctx.label.name + ".esm5"
+    if workspace:
+        out_dir = out_dir + "/" + workspace
+
+    outputs = [
+        ctx.actions.declare_file(_join([out_dir, _map_closure_path(f)]))
+        for f in target.typescript.replay_params.outputs
+        if not f.short_path.endswith(".externs.js")
+    ]
+
+    ctx.actions.run(
+        executable = ctx.executable._modify_tsconfig,
+        inputs = [target.typescript.replay_params.tsconfig],
+        outputs = [tsconfig],
+        arguments = [
+            target.typescript.replay_params.tsconfig.path,
+            tsconfig.path,
+            _join([workspace, target.label.package, ctx.label.name + ".esm5"]),
+            ctx.bin_dir.path,
+        ],
+    )
+
+    ctx.actions.run(
+        progress_message = "Compiling TypeScript (ES5 with ES Modules) %s" % target.label,
+        inputs = target.typescript.replay_params.inputs + [tsconfig],
+        outputs = outputs,
+        arguments = [tsconfig.path],
+        executable = target.typescript.replay_params.compiler,
+        execution_requirements = {
+            # TODO(alexeagle): enable worker mode for these compilations
+            "supports-workers": "0",
+        },
+    )
+
+    root_dir = _join([
+        ctx.bin_dir.path,
+        workspace,
+        target.label.package,
+        ctx.label.name + ".esm5",
+    ])
+
+    transitive_output = {root_dir: depset(outputs)}
+    for dep in ctx.rule.attr.deps:
+        if ESM5Info in dep:
+            transitive_output.update(dep[ESM5Info].transitive_output)
+
+    return [ESM5Info(
+        transitive_output = transitive_output,
+    )]
 
 # Downstream rules can use this aspect to access the ESM5 output flavor.
 # Only terminal rules (those which expect never to be used in deps[]) should do
 # this.
 esm5_outputs_aspect = aspect(
     implementation = _esm5_outputs_aspect,
     # Recurse to the deps of any target we visit
-    attr_aspects = ['deps'],
+    attr_aspects = ["deps"],
     attrs = {
         "_modify_tsconfig": attr.label(
             default = Label("//packages/bazel/src:modify_tsconfig"),
             executable = True,
-            cfg = "host"),
+            cfg = "host",
+        ),
         # We must list tsc_wrapped here to ensure it's built before the action runs
         # For some reason, having the compiler output as an input to the action above
         # is not sufficient.
@@ -128,43 +132,44 @@ esm5_outputs_aspect = aspect(
 )
 
 def esm5_root_dir(ctx):
-  return ctx.label.name + ".esm5"
+    return ctx.label.name + ".esm5"
 
 def flatten_esm5(ctx):
-  """Merge together the .esm5 folders from the dependencies.
-
-  Two different dependencies A and B may have outputs like
-  `bazel-bin/path/to/A.esm5/path/to/lib.js`
-  `bazel-bin/path/to/B.esm5/path/to/main.js`
-
-  In order to run rollup on this app, in case main.js contains `import from './lib'`
-  they need to be together in the same root directory, so if we depend on both A and B
-  we need the outputs to be
-  `bazel-bin/path/to/my_rule.esm5/path/to/lib.js`
-  `bazel-bin/path/to/my_rule.esm5/path/to/main.js`
-
-  Args:
-    ctx: the skylark rule execution context
-
-  Returns:
-    list of flattened files
-  """
-  esm5_sources = []
-  result = []
-  for dep in ctx.attr.deps:
-    if ESM5Info in dep:
-      transitive_output = dep[ESM5Info].transitive_output
-      esm5_sources.extend(transitive_output.values())
-  for f in depset(transitive = esm5_sources).to_list():
-    path = f.short_path[f.short_path.find(".esm5") + len(".esm5"):]
-    if (path.startswith("../")):
-      path = "external/" + path[3:]
-    rerooted_file = ctx.actions.declare_file("/".join([esm5_root_dir(ctx), path]))
-    result.append(rerooted_file)
-    # print("copy", f.short_path, "to", rerooted_file.short_path)
-    ctx.actions.expand_template(
-        output = rerooted_file,
-        template = f,
-        substitutions = {},
-    )
-  return result
+    """Merge together the .esm5 folders from the dependencies.
+
+    Two different dependencies A and B may have outputs like
+    `bazel-bin/path/to/A.esm5/path/to/lib.js`
+    `bazel-bin/path/to/B.esm5/path/to/main.js`
+
+    In order to run rollup on this app, in case main.js contains `import from './lib'`
+    they need to be together in the same root directory, so if we depend on both A and B
+    we need the outputs to be
+    `bazel-bin/path/to/my_rule.esm5/path/to/lib.js`
+    `bazel-bin/path/to/my_rule.esm5/path/to/main.js`
+
+    Args:
+      ctx: the skylark rule execution context
+
+    Returns:
+      list of flattened files
+    """
+    esm5_sources = []
+    result = []
+    for dep in ctx.attr.deps:
+        if ESM5Info in dep:
+            transitive_output = dep[ESM5Info].transitive_output
+            esm5_sources.extend(transitive_output.values())
+    for f in depset(transitive = esm5_sources).to_list():
+        path = f.short_path[f.short_path.find(".esm5") + len(".esm5"):]
+        if (path.startswith("../")):
+            path = "external/" + path[3:]
+        rerooted_file = ctx.actions.declare_file("/".join([esm5_root_dir(ctx), path]))
+        result.append(rerooted_file)
+
+        # print("copy", f.short_path, "to", rerooted_file.short_path)
+        ctx.actions.expand_template(
+            output = rerooted_file,
+            template = f,
+            substitutions = {},
+        )
+    return result