fix: fencing for npm packages (#946)

and other minor cleanups

Author: gregmagolan

internal/rollup/BUILD.bazel

@@ -87,7 +87,6 @@ nodejs_binary(
 )
 
 # BEGIN-INTERNAL
-# TODO: switch to npm_bazel_jasmine
 # buildozer: disable=load-on-top
 # buildozer: disable=out-of-order-load
 load("@npm_bazel_jasmine//:index.from_src.bzl", "jasmine_node_test")

packages/jasmine/BUILD.bazel

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-load("@build_bazel_rules_nodejs//:defs.bzl", "npm_package")
+load("@build_bazel_rules_nodejs//:defs.bzl", "COMMON_REPLACEMENTS", "npm_package")
 
 # Copy the license from our parent folder
 genrule(
@@ -40,6 +40,7 @@ npm_package(
     srcs = [
         "@npm_bazel_jasmine//:package_contents",
     ],
+    replacements = COMMON_REPLACEMENTS,
     vendor_external = [
         "npm_bazel_jasmine",
     ],

packages/jasmine/src/BUILD.bazel

@@ -1,3 +1,20 @@
+# Copyright 2019 The Bazel Authors. All rights reserved.
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+# BEGIN-INTERNAL
+# Parts of this BUILD file only necessary when building from source.
+# The generated `@bazel/jasmine` npm package contains a trimmed BUILD file using INTERNAL fences.
 load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
 load("@build_bazel_rules_nodejs//internal/js_library:js_library.bzl", "js_library")
 
@@ -31,3 +48,4 @@ js_library(
     ],
     module_name = "@bazel/jasmine",
 )
+# END-INTERNAL

packages/karma/src/BUILD.bazel

@@ -14,7 +14,7 @@
 
 # BEGIN-INTERNAL
 # Parts of this BUILD file only necessary when building from source.
-# The generated `@bazel/karma` npm package contains a trimmed BUILD file using # INTERNAL fences.
+# The generated `@bazel/karma` npm package contains a trimmed BUILD file using INTERNAL fences.
 load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
 load("@build_bazel_rules_nodejs//:defs.bzl", "nodejs_binary")
 load("@npm_bazel_typescript//:index.from_src.bzl", "ts_library")
@@ -82,4 +82,6 @@ filegroup(
 )
 
 # END-INTERNAL
-exports_files(["karma.conf.js"])
+exports_files([
+    "karma.conf.js",
+])

packages/labs/BUILD.bazel

@@ -1,4 +1,4 @@
-load("@build_bazel_rules_nodejs//:defs.bzl", "npm_package")
+load("@build_bazel_rules_nodejs//:defs.bzl", "COMMON_REPLACEMENTS", "npm_package")
 
 exports_files(["tsconfig.json"])
 
@@ -14,6 +14,7 @@ npm_package(
     srcs = [
         "@npm_bazel_labs//:package_contents",
     ],
+    replacements = COMMON_REPLACEMENTS,
     vendor_external = [
         "npm_bazel_labs",
     ],

packages/labs/src/BUILD.bazel

@@ -12,6 +12,9 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+# BEGIN-INTERNAL
+# Parts of this BUILD file only necessary when building from source.
+# The generated `@bazel/labs` npm package contains a trimmed BUILD file using INTERNAL fences.
 package(default_visibility = ["//visibility:public"])
 
 exports_files(["tsconfig.json"])
@@ -25,3 +28,4 @@ filegroup(
         "//webpack:package_contents",
     ],
 )
+# END-INTERNAL

packages/labs/src/webpack/BUILD.bazel

@@ -13,6 +13,8 @@
 # limitations under the License.
 
 # BEGIN-INTERNAL
+# Parts of this BUILD file only necessary when building from source.
+# The generated `@bazel/labs` npm package contains a trimmed BUILD file using INTERNAL fences.
 load("@build_bazel_rules_nodejs//:defs.bzl", "nodejs_binary")
 load("@npm_bazel_typescript//:index.from_src.bzl", "ts_library")
 

packages/protractor/BUILD.bazel

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-load("@build_bazel_rules_nodejs//:defs.bzl", "npm_package")
+load("@build_bazel_rules_nodejs//:defs.bzl", "COMMON_REPLACEMENTS", "npm_package")
 
 # Copy the license from our parent folder
 genrule(
@@ -40,6 +40,7 @@ npm_package(
     srcs = [
         "@npm_bazel_protractor//:package_contents",
     ],
+    replacements = COMMON_REPLACEMENTS,
     vendor_external = [
         "npm_bazel_protractor",
     ],

packages/stylus/BUILD.bazel

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-load("@build_bazel_rules_nodejs//:defs.bzl", "npm_package")
+load("@build_bazel_rules_nodejs//:defs.bzl", "COMMON_REPLACEMENTS", "npm_package")
 
 # Copy the license from our parent folder
 genrule(
@@ -40,6 +40,7 @@ npm_package(
     srcs = [
         "@npm_bazel_stylus//:package_contents",
     ],
+    replacements = COMMON_REPLACEMENTS,
     vendor_external = [
         "npm_bazel_stylus",
     ],

packages/stylus/src/BUILD.bazel

@@ -12,6 +12,9 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+# BEGIN-INTERNAL
+# Parts of this BUILD file only necessary when building from source.
+# The generated `@bazel/stylus` npm package contains a trimmed BUILD file using INTERNAL fences.
 load("@bazel_skylib//:bzl_library.bzl", "bzl_library")
 
 package(default_visibility = ["//visibility:public"])
@@ -33,3 +36,4 @@ filegroup(
         "package.json",
     ],
 )
+# END-INTERNAL