Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Add module owner to deno.land/x URLs #643

Closed
adelsz opened this issue May 20, 2020 · 3 comments
Closed

Add module owner to deno.land/x URLs #643

adelsz opened this issue May 20, 2020 · 3 comments

Comments

@adelsz
Copy link
Contributor

adelsz commented May 20, 2020

Having all module names in the global namespace leads to name squatting.
This is a proposal to add package owner name to deno.land/x URLs:

- https://deno.land/x/MODULE_NAME@BRANCH/SCRIPT.ts
+ https://deno.land/x/OWNER/MODULE_NAME@BRANCH/SCRIPT.ts

It should solve some of the problems raised in:
#547, #548, #406
I can create a PR implementing this if needed.

@alessandrorivero
Copy link

It should solve many of the problems raised in: #547, #548, #406

It only solves the problem of name squatting. All the other problems are still there. I think to really fix all problems there needs to be something like #406 (comment)

@adelsz
Copy link
Contributor Author

adelsz commented May 20, 2020

@alessandrorivero I agree, but #406 is closed now.
The whole database.json + PR mechanism definitely won't scale well, but might be working OK for the time being as ry mentioned.

This ticket proposes an immediate fix to name squatting and doesn't require much effort to implement.
I have changed the wording to solve some of the problems to avoid confusion.

@lucacasonato
Copy link
Member

This won't be done. If you want to scope by author, just import directly from GitHub.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants