-
Notifications
You must be signed in to change notification settings - Fork 38
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 support #28
Comments
What does deno need? |
You don't really need anything but mod.js file is welcome |
Cool. You're welcome to submit a pull request in case you don't want to wait for me to get around to this :) |
@marcelklehr I started to work on it so I added the file export * as topsort from './index.js'; And then I wanted to test if this work so I import it and got the error
There are a couple of options to solve this issue:
I prefer the 3rd option, which won't change the code and can prevent possible errors (which is critical cause many libraries depend on this) If you want I can update the README with instructions for how to use this library with Deno. Love to hear what you think |
Can you please add Deno support?
Since this project doesn't have any dependency it should be fairly easy
The text was updated successfully, but these errors were encountered: