refactor: deprecate npm_files list attribute in favor of npm_srcs depset

bazel-contrib / rules_nodejs

NodeJS toolchain for Bazel.

https://bazelbuild.github.io/rules_nodejs/

Apache License 2.0

724 stars 523 forks source link

refactor: deprecate npm_files list attribute in favor of npm_srcs depset #3738

Closed gregmagolan closed 5 months ago

gregmagolan commented 5 months ago

This simplifies downstream usage since npm_srcs can just be used directly instead of having to construct a depset from npm_files

The to_list() call is still necessary here in the toolchain creation but once the DEPRECATED fields can be removed then that will be go away. DEPRECATED fields can be removed in a future likely when support for WORKSPACE is dropped.