8000 Update dnt to solve library resolution issue by tobil4sk · Pull Request #68 · jacoborus/estilo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update dnt to solve library resolution issue #68

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 25, 2025
Merged

Update dnt to solve library resolution issue #68

merged 1 commit into from
Feb 25, 2025

Conversation

tobil4sk
Copy link
Contributor

Current version was causing this error:

Download https://registry-staging.deno.com/@std/path/meta.json
WARN Failed fetching https://registry-staging.deno.com/@std/path/meta.json. Retrying in 250ms...
WARN Failed fetching https://registry-staging.deno.com/@std/path/meta.json. Retrying in 500ms...
WARN Failed fetching https://registry-staging.deno.com/@std/path/meta.json. Retrying in 1000ms...
panicked at wasm\src\lib.rs:24:1:
unexpected exception: JsValue(TypeError: client error (Connect)
TypeError: client error (Connect)
    at async mainFetch (ext:deno_fetch/26_fetch.js:170:12)
    at async fetch (ext:deno_fetch/26_fetch.js:391:7)
    at async fetchWithRetries (https://deno.land/x/deno_cache@0.6.2/file_fetcher.ts:280:19)
    at async FileFetcher.#fetchRemote (https://deno.land/x/deno_cache@0.6.2/file_fetcher.ts:209:22)
    at async FileFetcher.fetch (https://deno.land/x/deno_cache@0.6.2/file_fetcher.ts:261:24))

See:

@hallvors
Copy link

I ran into this when trying to run the code from the official documentation. Tried both with Deno 1.45 and 2, was rather confused. Thank you for fixing.

@hallvors
Copy link

Same example code is also on the Deno blog: https://deno.com/blog/publish-esm-cjs-module-dnt - if docs need updating here please also fix the blog post. (I have not dived into whether the published code needs updating or not after all fixes have landed).

@jacoborus jacoborus merged commit 55fd027 into jacoborus:master Feb 25, 2025
@tobil4sk tobil4sk deleted the update-dnt branch February 26, 2025 01:56
@tobil4sk
Copy link
Contributor Author

if docs need updating here please also fix the blog post.

@hallvors Unfortunately, I haven't used deno much beyond estilo, so I don't know whether that deno blog should be updated. I guess someone should open an issue in the relevant repo: https://github.com/denoland/website_feedback/issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tobil4sk @hallvors @jacoborus
0