Skip to content
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.

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

Correctly show instantiated signatures for JSX element signature help and quick info #23492

Merged
merged 4 commits into from
May 16, 2018

Conversation

weswigham
Copy link
Member

@weswigham weswigham commented Apr 17, 2018

Fixes #22873
Fixes #22636

@weswigham weswigham changed the title Correctly show instantiated signatures for JSX element signature help Correctly show instantiated signatures for JSX element signature help and quick info Apr 18, 2018
@mhegazy
Copy link
Contributor

mhegazy commented May 16, 2018

@sandersn can you also take a look

@weswigham weswigham merged commit e01c7d2 into microsoft:master May 16, 2018
@weswigham
Copy link
Member Author

@mhegazy do I need to port this to the 2.9 branch?

@sandersn
Copy link
Member

I think this broke the build. It needs to use verify.signatureHelp({text: "..."}) instead of verify.SignatureHelpIs("...")

@weswigham weswigham deleted the jsx-signature-help branch May 16, 2018 20:47
@weswigham
Copy link
Member Author

weswigham commented May 16, 2018

Older PRs are great 😉 I've opened (and then merged) #24179 with the fix.

@ghost ghost mentioned this pull request May 17, 2018
@microsoft microsoft locked and limited conversation to collaborators Jul 31, 2018
This pull request was closed.
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.

3 participants