-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: upgrade deno_ast related crates #23187
Conversation
let source_map = if options.inline_source_map { | ||
SourceMapOption::Inline | ||
} else if options.source_map { | ||
SourceMapOption::Separate | ||
} else { | ||
SourceMapOption::None | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, the source map settings were so confusing...
@@ -159,6 +168,7 @@ pub async fn doc(flags: Flags, doc_flags: DocFlags) -> Result<(), AnyError> { | |||
kind_with_drilldown: | |||
deno_doc::html::DocNodeKindWithDrilldown::Other(node.kind), | |||
inner: std::sync::Arc::new(node), | |||
drilldown_parent_kind: None, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about this one. @crowlKats ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, thats correct
Not able to reproduce the crash locally on Linux. I sshed into the runner. Here's the stack trace & output:
The OOM is invoked. This looks like a memory bug in |
Also hitting the Vec capacity overflow sometimes:
|
**Description:** Fixes an incorrect capacity (found while investigating a segfault). More details: denoland/deno#23187 (comment)
This reverts commit 3afd2d3.
Maybe related: vercel/next.js#63924 Let's collect all debug information at swc-project/swc#8840 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, provided the tests pass
Had to revert back swc due to swc-project/swc#8840 Fixes: - denoland/deno_lint#1262 - denoland/deno_doc#538 - denoland/deno_doc#537 - denoland/deno_graph#430 - denoland/deno_graph#425 - denoland/deno_graph#432
Had to revert back swc due to swc-project/swc#8840 Fixes: - denoland/deno_lint#1262 - denoland/deno_doc#538 - denoland/deno_doc#537 - denoland/deno_graph#430 - denoland/deno_graph#425 - denoland/deno_graph#432
Had to revert back swc due to swc-project/swc#8840
Fixes: