-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
GitHub proxy via x (like "x/gh:githubusername:repo@tag") deprecated? #7151
Comments
Yes this is deprecated. We recently rolled out an update to the registry: https://deno.land/posts/registry2. This was incompatible with the |
Ah okay - no worries! Reading through that blog post it's clear the trade-off was worth it. |
It was previously possible to import a package from GitHub like this:
I can't find any notice about this being deprecated - I only noticed today when I tried to run an old-ish script. If it is deprecated, maybe this could be added as a message on the resulting 404 page? I'm happy to make a pull request for this. I'd just chance "This module does not exist." to something like "This module does not exist. The github proxy has been deprecated."
Also, if it was deprecated (i.e. this isn't a bug) was this discussed somewhere? I like deno's strong backwards-compatibility guarantees for the runtime and would hope that this philosophy spills over into other areas. That said, in these early days of deno a deprecation like this certainly has a lower cost and might be worth it.
The text was updated successfully, but these errors were encountered: