-
Notifications
You must be signed in to change notification settings - Fork 16
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
Provide npm package via dnt
#7
Conversation
It looks like the vento package name is already in use. @oscarotero what do you think? Regarding testing, unfortunately I didn't find the node.js fetch polyfill with Perhaps the dnt test should be disabled directly:: # ./scripts/build_npm.ts
await build({
+ test: false,
...
}) |
Hey, thanks for this. |
Okay, looks good. |
Almost done. btw, it might be possible to request a vento.js.org domain from js.org |
Sorry, I didn't explain well. I meant disable ONLY this test in Node (but run all other tests). You can use the Deno.test({
name: "Print async filters",
ignore: typeof Deno === "undefined"
fn: async () => {...}
});
That would be great. I'm a bit busy right now but feel free to request it.
Yeah, sure! I just send you an invitation. |
Ok, but
Simply put, a
thx! |
thanks! |
Hmmm... It looks like the github pages for this repository are not deployed in the same way. |
Ok, done |
It is currently stuck in the "Print async filters" test of
print.test.ts
.undici for node.js does not support
file://
url, will get a "not implemented yet" error (nodejs/undici#2144)Run the dnt build:
deno run -A scripts/build_npm.ts <version>