-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Feature Request: Deno Support #1442
Comments
What would be required to make this work? |
Closing this due to an apparent lack of interest. Happy to revisit should someone be able to lead this effort. |
I'm interested in this too. My gut feeling tells me that it shouldn't be too complicated because for an earlier version of cheerio someone already made it to work: https://github.com/justjavac/deno_cheerio. That said, I would need some support on this because I'm fairly new to both deno and cheerio. |
Just reading the diffs in the README (as that is the only part I can read), this seems to be primarily an issue with the use of the |
Cheerio works in Deno with import cheerio from "https://dev.jspm.io/npm:cheerio/index.js";
const $ = cheerio.load('<h2 class="title">Hello world</h2>');
$("h2.title").text("Hello Deno!");
$("h2").addClass("deno");
console.log($.html()); Run it: > deno run --no-check mod.ts
<h2 class="title deno">Hello Deno!</h2> What is not compatible with deno is the dts |
Now Cheerio also works in Deno without import cheerio from "https://esm.sh/cheerio@1.0.0-rc.10";
const $ = cheerio.load('<h2 class="title">Hello world</h2>');
$("h2.title").text("Hello Deno!");
$("h2").addClass("deno");
console.log($.html()); > deno run mod.ts
<html><head></head><body><h2 class="title deno">Hello Deno!</h2></body></html> |
Any plans to support Deno?
https://deno.land/
The text was updated successfully, but these errors were encountered: