Skip to content

Document banned npm names#381

Merged
sandersn merged 2 commits intomicrosoft:masterfrom
LinqLover:document-banned-names
Jan 25, 2022
Merged

Document banned npm names#381
sandersn merged 2 commits intomicrosoft:masterfrom
LinqLover:document-banned-names

Conversation

@LinqLover
Copy link
Contributor

@LinqLover LinqLover commented Jan 15, 2022

See also microsoft/dtslint#349 and microsoft/dtslint#350. As soon as an official decision for this issue exists, that should be referenced in the docs as well.

Cherry-picked from microsoft/dtslint#351.

@sandersn
Copy link
Member

I sent an internal request to the npm team to exempt the @types scope from name checks. I haven't heard back but best-case is that it gets added as a low-priority work item to be fixed some time in the future.

@sandersn sandersn merged commit 4d87da0 into microsoft:master Jan 25, 2022
@LinqLover
Copy link
Contributor Author

Nice, thank you! It's a pity that we can't just send a PR ready-to-merge, though ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants