Skip to content

Commit

Permalink
fix(builtin): add DeclarationInfo sources from dependencies as inputs…
Browse files Browse the repository at this point in the history
… to npm_package_bin driven actions (#2353)
  • Loading branch information
mattem authored Dec 16, 2020
1 parent c148138 commit 85dc132
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion internal/node/npm_package_bin.bzl
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
"A generic rule to run a tool that appears in node_modules/.bin"

load("//:providers.bzl", "JSModuleInfo", "NpmPackageInfo", "node_modules_aspect", "run_node")
load("//:providers.bzl", "DeclarationInfo", "JSModuleInfo", "NpmPackageInfo", "node_modules_aspect", "run_node")
load("//internal/common:expand_variables.bzl", "expand_variables")
load("//internal/linker:link_node_modules.bzl", "module_mappings_aspect")

Expand Down Expand Up @@ -39,6 +39,8 @@ def _inputs(ctx):
inputs_depsets.append(d[NpmPackageInfo].sources)
if JSModuleInfo in d:
inputs_depsets.append(d[JSModuleInfo].sources)
if DeclarationInfo in d:
inputs_depsets.append(d[DeclarationInfo].declarations)
return depset(ctx.files.data, transitive = inputs_depsets).to_list()

def _impl(ctx):
Expand Down

0 comments on commit 85dc132

Please sign in to comment.