-
Notifications
You must be signed in to change notification settings - Fork 9
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
Use Deno #5
Comments
Something like this would be cool
|
Isn't running a script from a url generally frowned on tho? |
not with It's probably more secure than running
|
Awesome, I didn't know that about deno. :) |
The first step will be to create a deno version of the CLI. Node introduced the convention of conditional exports. And skypack seems to support exports for different environments such as deno, they mention it at https://docs.skypack.dev/package-authors/package-checks#export-map Changing {
"exports": "./bin/octoherd.js"
} to {
"exports": {
"node": "./bin/octoherd-node.js",
"node": "./bin/octoherd-deno.js"
}
} Maybe we can also make the current CLI just work with Deno out of the box, but I'm not familiar enough with Deno to debug the errors I currently see. Once we have a valid Deno export and Skypack supports it, a script such as the above mentiond
I'd love help with this. |
I think the CLI would be a great use case to use Deno. Scripts and dependencies could be loaded directly from a remote URL, and Deno has a better security system.
The text was updated successfully, but these errors were encountered: