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

Deno runtime support #457

Open
3 tasks
donmccurdy opened this issue Dec 23, 2021 · 2 comments
Open
3 tasks

Deno runtime support #457

donmccurdy opened this issue Dec 23, 2021 · 2 comments
Labels
feature New enhancement or request package:core

Comments

@donmccurdy
Copy link
Owner

donmccurdy commented Dec 23, 2021

Continuing from #380 and #379, I intend to support Deno as an environment. The I/O class has been added, but a few remaining things to do:

Context:

TypeScript's magic ./foo./foo.ts resolution is causing quite a headache here.

@donmccurdy donmccurdy added feature New enhancement or request package:core labels Dec 23, 2021
@donmccurdy donmccurdy added this to the v2.0 milestone Dec 23, 2021
This was referenced Dec 23, 2021
@donmccurdy donmccurdy modified the milestones: v2.0, Backlog Jan 29, 2022
@donmccurdy donmccurdy changed the title Support Deno environment Deno runtime support Oct 25, 2023
@rotu
Copy link
Contributor

rotu commented Oct 15, 2024

It seems like DenoIO.write doesn't exist and NodeIO works fine on deno. Maybe DenoIO isn't needed?

@donmccurdy
Copy link
Owner Author

I'm not caught up on the latest changes in Deno v2 yet, but I understand they've prioritized compatibility with the Node.js ecosystem, and I agree that a DenoIO class isn't strictly required. That said — I think I do prefer to keep it, even if it's just an empty subclass of NodeIO. Deno isn't Node.js, and I think deserves equal standing as an I/O class. It's possible we may want a slightly different implementation at some point, too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New enhancement or request package:core
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants