diff options
| author | David Sherret <dsherret@users.noreply.github.com> | 2024-07-09 12:15:03 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-07-09 12:15:03 -0400 |
| commit | 839caf6fafdf9ca1cdec6cd9cef38296be41145f (patch) | |
| tree | 691dba21b45e9c5640275304308aa5d8a5d4a7ba /cli/worker.rs | |
| parent | 07613a6bf26d9112d47fda9e502425395bd78105 (diff) | |
refactor: use concrete error types for node resolution (#24470)
This will help clean up some of the code in the CLI because we'll be
able to tell how the resolution failed (not part of this PR).
Diffstat (limited to 'cli/worker.rs')
| -rw-r--r-- | cli/worker.rs | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/cli/worker.rs b/cli/worker.rs index 987d65192..9125f28be 100644 --- a/cli/worker.rs +++ b/cli/worker.rs @@ -669,7 +669,7 @@ impl CliMainWorkerFactory { self.resolve_binary_entrypoint_fallback(package_folder, sub_path); match result { Ok(Some(resolution)) => Ok(resolution), - Ok(None) => Err(original_err), + Ok(None) => Err(original_err.into()), Err(fallback_err) => { bail!("{:#}\n\nFallback failed: {:#}", original_err, fallback_err) } @@ -692,16 +692,13 @@ impl CliMainWorkerFactory { return Ok(None); } - // use a fake referrer since a real one doesn't exist - let referrer = - ModuleSpecifier::from_directory_path(package_folder).unwrap(); let Some(resolution) = self .shared .node_resolver .resolve_package_subpath_from_deno_module( package_folder, sub_path, - &referrer, + /* referrer */ None, NodeResolutionMode::Execution, )? else { |