diff options
| author | David Sherret <dsherret@users.noreply.github.com> | 2024-07-25 09:07:59 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-07-25 09:07:59 -0400 |
| commit | 763f05e74dfd0032b238603f625893a52e363591 (patch) | |
| tree | c6a71559472755919358afa53eecac206cad80a9 /cli/tools/registry | |
| parent | ef78d317f084ffe633253acd138a48a425113fa7 (diff) | |
fix(unstable): move sloppy-import warnings to lint rule (#24710)
Adds a new `no-sloppy-imports` lint rule and cleans up the lint code.
Closes #22844
Closes https://github.com/denoland/deno_lint/issues/1293
Diffstat (limited to 'cli/tools/registry')
| -rw-r--r-- | cli/tools/registry/mod.rs | 4 | ||||
| -rw-r--r-- | cli/tools/registry/unfurl.rs | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/cli/tools/registry/mod.rs b/cli/tools/registry/mod.rs index c9b742588..586327821 100644 --- a/cli/tools/registry/mod.rs +++ b/cli/tools/registry/mod.rs @@ -45,7 +45,7 @@ use crate::graph_util::ModuleGraphCreator; use crate::http_util::HttpClient; use crate::resolver::SloppyImportsResolver; use crate::tools::check::CheckOptions; -use crate::tools::lint::no_slow_types; +use crate::tools::lint::collect_no_slow_type_diagnostics; use crate::tools::registry::diagnostics::PublishDiagnostic; use crate::tools::registry::diagnostics::PublishDiagnosticsCollector; use crate::util::display::human_size; @@ -341,7 +341,7 @@ impl PublishPreparer { for package in package_configs { let export_urls = package.config_file.resolve_export_value_urls()?; let diagnostics = - no_slow_types::collect_no_slow_type_diagnostics(&export_urls, &graph); + collect_no_slow_type_diagnostics(&graph, &export_urls); if !diagnostics.is_empty() { any_pkg_had_diagnostics = true; for diagnostic in diagnostics { diff --git a/cli/tools/registry/unfurl.rs b/cli/tools/registry/unfurl.rs index f7c1049ca..a28ba445a 100644 --- a/cli/tools/registry/unfurl.rs +++ b/cli/tools/registry/unfurl.rs @@ -177,8 +177,8 @@ impl SpecifierUnfurler { if let Some(sloppy_imports_resolver) = &self.sloppy_imports_resolver { sloppy_imports_resolver .resolve(&resolved, deno_graph::source::ResolutionMode::Execution) - .as_specifier() - .clone() + .map(|res| res.into_specifier()) + .unwrap_or(resolved) } else { resolved }; |
