Skip to content

Conversation

@sandersn
Copy link
Member

@sandersn sandersn commented Aug 7, 2020

Fixes bogus deprecated notices on imports of functions with deprecated overloads, but with some non-deprecated overloads.

This should ship in 4.0 too.

Fixes microsoft/vscode#104238

Fixes bogus deprecated notices on imports of functions with deprecated
overloads, but with some non-deprecated overloads.

Fixes microsoft/vscode#104238
@typescript-bot typescript-bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Aug 7, 2020
@sandersn
Copy link
Member Author

sandersn commented Aug 7, 2020

@Kingwl you probably want to see this too.

Copy link
Contributor

@Kingwl Kingwl left a comment

Choose a reason for hiding this comment

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

lgtm

@sandersn sandersn merged commit 3328fdb into master Aug 7, 2020
@sandersn
Copy link
Member Author

sandersn commented Aug 7, 2020

@typescript-bot cherry-pick this to release-4.0

@typescript-bot
Copy link
Collaborator

typescript-bot commented Aug 7, 2020

Heya @sandersn, I've started to run the task to cherry-pick this into release-4.0 on this PR at e2eb319. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

Hey @sandersn, I've opened #39951 for you.

typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request Aug 7, 2020
Component commits:
6e5d75b Use isUncalledFunctionReference for aliases too
Fixes bogus deprecated notices on imports of functions with deprecated
overloads, but with some non-deprecated overloads.

Fixes microsoft/vscode#104238

e2eb319 Just check all declarations, don't call isUncalledFunction
sandersn added a commit that referenced this pull request Aug 7, 2020
Component commits:
6e5d75b Use isUncalledFunctionReference for aliases too
Fixes bogus deprecated notices on imports of functions with deprecated
overloads, but with some non-deprecated overloads.

Fixes microsoft/vscode#104238

e2eb319 Just check all declarations, don't call isUncalledFunction

Co-authored-by: Nathan Shively-Sanders <nathansa@microsoft.com>
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Question about ts/js deprecated hint

5 participants