diff options
author | David Sherret <dsherret@users.noreply.github.com> | 2023-09-29 09:26:25 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-29 09:26:25 -0400 |
commit | 5edd102f3f912a53c7bcad3b0fa4feb672ada323 (patch) | |
tree | 2402b64e527bd859f3a2c71d3e96a89992002aa2 /cli/ops | |
parent | d43e48c4e96b02289d505cd2558ba85d7d6cb57b (diff) |
refactor(cli): make `CliNpmResolver` a trait (#20732)
This makes `CliNpmResolver` a trait. The terminology used is:
- **managed** - Deno manages the node_modules folder and does an
auto-install (ex. `ManagedCliNpmResolver`)
- **byonm** - "Bring your own node_modules" (ex. `ByonmCliNpmResolver`,
which is in this PR, but unimplemented at the moment)
Part of #18967
Diffstat (limited to 'cli/ops')
-rw-r--r-- | cli/ops/mod.rs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/cli/ops/mod.rs b/cli/ops/mod.rs index ab3a55468..eb75dc272 100644 --- a/cli/ops/mod.rs +++ b/cli/ops/mod.rs @@ -12,7 +12,7 @@ pub mod bench; pub mod jupyter; pub mod testing; -pub fn cli_exts(npm_resolver: Arc<CliNpmResolver>) -> Vec<Extension> { +pub fn cli_exts(npm_resolver: Arc<dyn CliNpmResolver>) -> Vec<Extension> { vec![ #[cfg(not(feature = "__runtime_js_sources"))] cli::init_ops(npm_resolver), @@ -33,7 +33,7 @@ deno_core::extension!(cli, "99_main.js" ], options = { - npm_resolver: Arc<CliNpmResolver>, + npm_resolver: Arc<dyn CliNpmResolver>, }, state = |state, options| { state.put(options.npm_resolver); @@ -51,6 +51,6 @@ deno_core::extension!(cli, #[op2] #[string] fn op_npm_process_state(state: &mut OpState) -> Result<String, AnyError> { - let npm_resolver = state.borrow_mut::<Arc<CliNpmResolver>>(); + let npm_resolver = state.borrow_mut::<Arc<dyn CliNpmResolver>>(); Ok(npm_resolver.get_npm_process_state()) } |