Skip to content

Commit 6ff6fa1

Browse files
committed
use git deno_graph
1 parent ef2349f commit 6ff6fa1

5 files changed

Lines changed: 41 additions & 43 deletions

File tree

Cargo.lock

Lines changed: 3 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ repository = "https://github.com/denoland/deno_doc"
1111
members = ["lib"]
1212

1313
[workspace.dependencies]
14-
deno_graph = { path = "../deno_graph", default-features = false, features = [
14+
deno_graph = { git = "https://github.com/nathanwhit/deno_graph", branch = "oxc-port", default-features = false, features = [
1515
"swc",
1616
"symbols",
1717
] }
@@ -20,9 +20,6 @@ import_map = "0.25.0"
2020
serde = { version = "1.0.204", features = ["derive"] }
2121
serde_json = { version = "1.0.122", features = ["preserve_order"] }
2222

23-
[patch."https://github.com/nathanwhit/deno_ast"]
24-
deno_ast = { path = "../deno_ast" }
25-
2623
[lib]
2724
name = "deno_doc"
2825

examples/ddoc/main.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -259,13 +259,13 @@ async fn run() -> anyhow::Result<()> {
259259
// This allows comparing renamed modules.
260260
let old_by_original = old_specifiers
261261
.iter()
262-
.zip(old_docs.into_iter())
262+
.zip(old_docs)
263263
.map(|(orig, (_, nodes))| (orig.clone(), nodes))
264264
.collect::<IndexMap<_, _>>();
265265

266266
let new_by_original = new_specifiers
267267
.iter()
268-
.zip(new_docs.into_iter())
268+
.zip(new_docs)
269269
.map(|(orig, (_, nodes))| (orig.clone(), nodes))
270270
.collect::<IndexMap<_, _>>();
271271

src/html/mod.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ use serde::Serialize;
1010
use std::borrow::Cow;
1111
use std::cmp::Ordering;
1212
use std::collections::HashMap;
13+
use std::path::Path;
1314
use std::path::PathBuf;
1415
use std::sync::Arc;
1516

@@ -1766,8 +1767,7 @@ pub fn find_common_ancestor<'a>(
17661767
}
17671768
}
17681769

1769-
if common_ancestor.as_os_str().is_empty()
1770-
|| common_ancestor == PathBuf::from("/")
1770+
if common_ancestor.as_os_str().is_empty() || common_ancestor == Path::new("/")
17711771
{
17721772
None
17731773
} else {

src/parser.rs

Lines changed: 33 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -565,41 +565,40 @@ impl<'a> DocParser<'a> {
565565
for stmt in module_info.statements() {
566566
if let Statement::ImportDeclaration(import_decl) = stmt
567567
&& let Some(js_doc) = js_doc_for_range(module_info, import_decl.span)
568+
&& let Some(specifiers) = &import_decl.specifiers
568569
{
569-
if let Some(specifiers) = &import_decl.specifiers {
570-
for specifier in specifiers {
571-
let (imported_name, original_name, src) = match specifier {
572-
ImportDeclarationSpecifier::ImportSpecifier(named_specifier) => (
573-
named_specifier.local.name.to_string(),
574-
Some(module_export_name_value(&named_specifier.imported)),
575-
&import_decl.source.value,
576-
),
577-
ImportDeclarationSpecifier::ImportDefaultSpecifier(
578-
default_specifier,
579-
) => (
580-
default_specifier.local.name.to_string(),
581-
Some("default".to_string()),
582-
&import_decl.source.value,
583-
),
584-
ImportDeclarationSpecifier::ImportNamespaceSpecifier(
585-
namespace_specifier,
586-
) => (
587-
namespace_specifier.local.name.to_string(),
588-
None,
589-
&import_decl.source.value,
590-
),
591-
};
592-
593-
let resolved_specifier =
594-
self.resolve_dependency(&src.to_string(), referrer)?;
595-
596-
imports.push(Import {
597-
imported_name: imported_name.into_boxed_str(),
598-
js_doc: js_doc.clone(),
599-
src: resolved_specifier.to_string(),
600-
original_name,
601-
});
602-
}
570+
for specifier in specifiers {
571+
let (imported_name, original_name, src) = match specifier {
572+
ImportDeclarationSpecifier::ImportSpecifier(named_specifier) => (
573+
named_specifier.local.name.to_string(),
574+
Some(module_export_name_value(&named_specifier.imported)),
575+
&import_decl.source.value,
576+
),
577+
ImportDeclarationSpecifier::ImportDefaultSpecifier(
578+
default_specifier,
579+
) => (
580+
default_specifier.local.name.to_string(),
581+
Some("default".to_string()),
582+
&import_decl.source.value,
583+
),
584+
ImportDeclarationSpecifier::ImportNamespaceSpecifier(
585+
namespace_specifier,
586+
) => (
587+
namespace_specifier.local.name.to_string(),
588+
None,
589+
&import_decl.source.value,
590+
),
591+
};
592+
593+
let resolved_specifier =
594+
self.resolve_dependency(src.as_ref(), referrer)?;
595+
596+
imports.push(Import {
597+
imported_name: imported_name.into_boxed_str(),
598+
js_doc: js_doc.clone(),
599+
src: resolved_specifier.to_string(),
600+
original_name,
601+
});
603602
}
604603
}
605604
}

0 commit comments

Comments
 (0)