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.

Sign up for GitHub

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

Jump to bottom

Update fixer for TS#56571 to remove all References #8

Merged

NullVoxPopuli merged 1 commit into main from remove-every-reference-by-default

Feb 9, 2024

Owner

NullVoxPopuli commented Feb 9, 2024

microsoft/TypeScript#56571 (comment)

I'm increasingly starting to believe /// <reference is not the correct approach.

You

move types to dependencies
or declare a peerDependencies entry on the types
and your consumer provides the types (this is the Scenario the TS team has been hesitant to support)


          Update tests

500df38

NullVoxPopuli added the bug label

NullVoxPopuli merged commit 56c1b08 into main

5 checks passed

NullVoxPopuli deleted the remove-every-reference-by-default branch

February 9, 2024 05:40

github-actions bot mentioned this pull request

Prepare Release #9

Merged

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

Labels

bug