Skip to content
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

Obligatory Deno question #77

Open
vegerot opened this issue Sep 25, 2021 · 6 comments
Open

Obligatory Deno question #77

vegerot opened this issue Sep 25, 2021 · 6 comments
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers

Comments

@vegerot
Copy link

vegerot commented Sep 25, 2021

It seems hard to go into a JavaScript/TypeScript library's issues and not have someone asking about Deno

Well, here it is: does typescript-coverage-report support Deno?

@vegerot vegerot added the enhancement New feature or request label Sep 25, 2021
@alexcanessa
Copy link
Owner

Hello @vegerot sorry didn't see this.
As far as I understand Deno is for runtime, where this tool tests and generates type coverage of your project.

Am I missing something here?

@vegerot
Copy link
Author

vegerot commented Dec 21, 2021

Sorry, I should have explained more clearly. In the Install instructions you give instructions to install a Node.js package, but no Deno instructions. I would like something like

deno install --allow-read https://alexcanessa.com/typescript-coverage-report.ts

or something 🙂

I'll probably open an issue on https://github.com/plantain-00/type-coverage too to ask about a Deno API for that too 😁

Thanks

@alexcanessa
Copy link
Owner

Ok ok, so just a README.md update? Fancy contributing too? 🎉

@alexcanessa alexcanessa added documentation Improvements or additions to documentation good first issue Good for newcomers and removed enhancement New feature or request labels Dec 22, 2021
@vegerot
Copy link
Author

vegerot commented Dec 22, 2021

Just a README.md update if that's all that's needed. Does this package use any Node libraries (like fs, path, etc.)?

Does running the above command work for you?

@alexcanessa
Copy link
Owner

@vegerot yes it does use both fs and path.
Are you able to check yourself? I'm assuming you got Deno?

@vegerot
Copy link
Author

vegerot commented Dec 27, 2021

Well first of all, I'm not even sure which URL I should use to install it.

https://deno.land/manual/tools/script_installer#script-installer

Should I try unpkg?

Note that Deno does not have any Node.js libraries, so using fs or path will not work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants