Skip to content

signaturehelper could handle rest union type. #39294

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

Closed
wants to merge 6 commits into from

Conversation

ShuiRuTian
Copy link
Contributor

Fixes #39227

@weswigham
Copy link
Member

I am so sorry for this (normally I'd review and iterate on this to merge it); but I've put up #39317 with a more minimal change (as the change required to fix the bug really is quite small) - I already had it ready and didn't see someone'd already put up another PR to work on the bug until I went to post the PR. The approach is a little different than the one taken here.

@ShuiRuTian
Copy link
Contributor Author

ShuiRuTian commented Jun 29, 2020

@weswigham Oh, no need to be sorry, I jsut see your fix throgh the notification, and I also think it is better~

@ShuiRuTian ShuiRuTian closed this Jun 29, 2020
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.

Union of unlabled tuple rest args have bad signature help parameter names
2 participants