deno
deno copied to clipboard
`deno add` - support `http:` and `https:` specifiers
Currently deno add only works with jsr: and npm: specifiers.
We should add an ability to do deno add https://deno.land/x/foo/bar.ts.
To do this, we should leverage existing infer_name_from_url() helper, that is already used in deno install and deno compile
https://github.com/denoland/deno/blob/d3d3582b1f1b71b18d8d89cdf8e14f112b8f40a7/cli/tools/installer.rs#L131-L185
Note that there is #23144 for aliases
This would require modifying https://github.com/denoland/deno_semver.
@tristanisham why?
@tristanisham why?
I believe that's where all the code for specifiers is. If the goal is to add "http(s):" that's the package where the actual structs and implementation details reside.
For the record:
Had a discussion about adding the feature to the deno_semver crate, but concluded that it would be better implemented here.
@bartlomieju @tristanisham Can I take this issue? I may have some questions regarding certain internals of the project but I would like to give it a shot c:
Be my guess. I haven't finished my PR.
Hi @jmj0502, @tristanisham @bartlomieju , if you guys are not working on this then can I pick this up please.
Sure, feel free to pick it up.