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

Exclude download-git-repo from forbidden words #349

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LinqLover
Copy link

This package already exists on npm, so if I understand the matter correctly, it should be allowed here, too:

https://www.npmjs.com/package/download-git-repo

Complements ad7eb60. Required for DefinitelyTyped/DefinitelyTyped#56874.

This package already exists on npm, so if I understand the matter correctly, it should be allowed here, too:

https://www.npmjs.com/package/download-git-repo

Complements ad7eb60. Required for DefinitelyTyped/DefinitelyTyped#56874.
Copy link
Member

@sandersn sandersn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, no. =( Even though download-git-repo is allowed on npm because it's old, DT is not allowed to publish @types/download-git-repo, because it will be a new package.

@LinqLover
Copy link
Author

Ah, I see, so what does this mean for my PR to DefinitelyTyped (DefinitelyTyped/DefinitelyTyped#56874) Do I have to submit it under a different name?

LinqLover added a commit to LinqLover/dtslint that referenced this pull request Dec 22, 2021
See also microsoft#349 and microsoft#350. As soon as an official decision for this issue exists, that should be referenced in the docs as well.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants