Fix the confusing `_lib` names in //scouting
After helping Wesley with a question, I realized that calling the
libraries `_lib` is not helpful. Instead, we should just use the names
specified in the BUILD files.
Signed-off-by: Philipp Schrader <philipp.schrader@gmail.com>
Change-Id: I936ba1316fd1d38cc592fd512aeea8e50411c6c4
diff --git a/tools/build_rules/js.bzl b/tools/build_rules/js.bzl
index ceb67aa..eb95510 100644
--- a/tools/build_rules/js.bzl
+++ b/tools/build_rules/js.bzl
@@ -1,7 +1,5 @@
load("@aspect_rules_js//js:providers.bzl", "JsInfo")
load("@bazel_skylib//rules:write_file.bzl", "write_file")
-load("@aspect_rules_js//js:defs.bzl", "js_library")
-load("@aspect_rules_js//npm:defs.bzl", "npm_package")
load("@aspect_bazel_lib//lib:copy_to_directory.bzl", "copy_to_directory")
load("@aspect_bazel_lib//lib:copy_file.bzl", "copy_file")
load("@aspect_rules_esbuild//esbuild:defs.bzl", "esbuild")
@@ -271,7 +269,7 @@
srcs.append(":_public_api")
ng_project(
- name = "_lib",
+ name = name,
srcs = srcs + [":_index"],
deps = deps + PACKAGE_DEPS,
#visibility = ["//visibility:private"],
@@ -279,20 +277,6 @@
**kwargs
)
- js_library(
- name = name + "_js",
- srcs = [":_lib"],
- visibility = ["//visibility:public"],
- )
-
- npm_package(
- name = name,
- srcs = ["package.json", ":_lib"],
- # This is a perf improvement; the default will be flipped to False in rules_js 2.0
- include_runfiles = False,
- visibility = ["//visibility:public"],
- )
-
def rollup_bundle(name, entry_point, deps = [], visibility = None, **kwargs):
"""Calls the upstream rollup_bundle() and exposes a .min.js file.