Skip to content

Commit

Permalink
refactor: remove deprecated update_pnpm_lock_node_toolchain_prefix fr…
Browse files Browse the repository at this point in the history
…om npm_translate_lock (#1574)
  • Loading branch information
gregmagolan committed Apr 20, 2024
1 parent e40cc44 commit 5be4bd7
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions npm/private/npm_translate_lock.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -555,9 +555,6 @@ def npm_translate_lock(
**kwargs: Internal use only
"""

# TODO(2.0): remove backward compat support for update_pnpm_lock_node_toolchain_prefix
update_pnpm_lock_node_toolchain_prefix = kwargs.pop("update_pnpm_lock_node_toolchain_prefix", None)

# TODO(2.0): move this to a new required rules_js_repositories() WORKSPACE function
if register_copy_directory_toolchains and not native.existing_rule("copy_directory_toolchains"):
_register_copy_directory_toolchains()
Expand Down Expand Up @@ -659,7 +656,7 @@ def npm_translate_lock(
data = data,
preupdate = preupdate,
quiet = quiet,
node_toolchain_prefix = update_pnpm_lock_node_toolchain_prefix if update_pnpm_lock_node_toolchain_prefix else node_toolchain_prefix,
node_toolchain_prefix = node_toolchain_prefix,
use_pnpm = use_pnpm,
yq_toolchain_prefix = yq_toolchain_prefix,
npm_package_target_name = npm_package_target_name,
Expand Down

0 comments on commit 5be4bd7

Please sign in to comment.