Add missing progress_message to actions

Fixes bazel-contrib#705
alexeagle · Apr 20, 2019 · 0ef6ea9 · 0ef6ea9
1 parent b4d4bbf
commit 0ef6ea9
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 2 deletions.
diff --git a/examples/parcel/parcel.bzl b/examples/parcel/parcel.bzl
@@ -36,7 +36,7 @@ def _parcel_impl(ctx):
         executable = ctx.executable.parcel,
         outputs = [ctx.outputs.bundle, ctx.outputs.sourcemap],
         arguments = args,
-        progress_message = "Running Parcel to produce %s" % ctx.outputs.bundle.path,
+        progress_message = "Bundling JavaScript %s [parcel]" % ctx.outputs.bundle.short_path,
     )
     return [DefaultInfo()]
 

diff --git a/internal/npm_package/npm_package.bzl b/internal/npm_package/npm_package.bzl
@@ -66,6 +66,7 @@ def create_package(ctx, deps_sources, nested_packages):
         inputs.append(ctx.version_file)
 
     ctx.actions.run(
+        progress_message = "Assembling npm package %s" % package_dir.short_path,
         executable = ctx.executable._packager,
         inputs = inputs,
         outputs = [package_dir, ctx.outputs.pack, ctx.outputs.publish],

diff --git a/internal/rollup/rollup_bundle.bzl b/internal/rollup/rollup_bundle.bzl
@@ -213,6 +213,7 @@ def _run_rollup(ctx, sources, config, output, map_output = None):
         outputs += [map_output]
 
     ctx.actions.run(
+        progress_message = "Bundling JavaScript %s [rollup]" % output.short_path,
         executable = ctx.executable._rollup,
         inputs = depset(direct_inputs, transitive = [sources]),
         outputs = outputs,
@@ -227,6 +228,7 @@ def _run_tsc(ctx, input, output):
     args.add_all(["--outFile", output.path])
 
     ctx.actions.run(
+        progress_message = "Downleveling JavaScript to ES5 %s [typescript]" % output.short_path,
         executable = ctx.executable._tsc,
         inputs = [input],
         outputs = [output],
@@ -242,6 +244,7 @@ def _run_tsc_on_directory(ctx, input_dir, output_dir):
     args.add_all(["--output", output_dir.path])
 
     ctx.actions.run(
+        progress_message = "Downleveling JavaScript to ES5 %s [typescript]" % output_dir.short_path,
         executable = ctx.executable._tsc_directory,
         inputs = [input_dir],
         outputs = [output_dir, config],
@@ -314,6 +317,7 @@ def _run_terser(ctx, input, output, map_output, debug = False, comments = True,
         args.add("--beautify")
 
     ctx.actions.run(
+        progress_message = "Optimizing JavaScript %s [terser]" % output.short_path,
         executable = ctx.executable._terser_wrapped,
         inputs = inputs,
         outputs = outputs,

diff --git a/packages/labs/webpack/src/webpack_bundle.bzl b/packages/labs/webpack/src/webpack_bundle.bzl
@@ -19,7 +19,7 @@ def _webpack_bundle(ctx):
         executable = ctx.executable.webpack,
         outputs = [ctx.outputs.bundle, ctx.outputs.sourcemap],
         arguments = [args],
-        progress_message = "Bundling with Webpack: %s" % ctx.outputs.bundle.path,
+        progress_message = "Bundling JavaScript %s [webpack]" % ctx.outputs.bundle.path,
     )
     return [DefaultInfo()]