-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
thread 'tokio-runtime-worker' panicked at /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/deno_graph-0.69.6/src/graph.rs:1525:63: called Option::unwrap()
on a None
value
#22852
Labels
bug
Something isn't working correctly
Comments
Here it is with
|
You can temporarily work around this by using |
nathanwhit
pushed a commit
that referenced
this issue
Mar 14, 2024
In addition to the reasons for this outlined by @nayeemrmn in #14877 (which I think are reasons alone to not do this), this simplifies things a lot because then we don't need to implement the following: 1. Need to handle a JSR module dynamically importing a module within it. 2. Need to handle importing an export of a JSR dep then another export dynamically loaded later. Additionally, people should be running `deno check dynamic_import.ts` instead of relying on this behaviour. Landing this as a fix because it's blocking people in some scenarios and the current behaviour is broken (I didn't even have to change any tests to remove this, which is bad). Closes #22852 Closes #14877 Closes #22580
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Check CI output here: https://gitlab.com/soapbox-pub/mostr/-/jobs/6369700712
The text was updated successfully, but these errors were encountered: