-
Notifications
You must be signed in to change notification settings - Fork 12.2k
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
Port Extract Method fixes from master #18518
Conversation
@amcasey, |
Hold off on reviewing this - I have to rebase away #18516. |
68ae415
to
30ac684
Compare
I dropped everything about type parameters, but kept the fourslash changes to make merging easier. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to remove the failing test file for now.
Turns out it was merged in in 387fc20. I'll rebase and fix it. |
Partial port of ea4fdc5
Check `scope.name` when trying to extract from an anon class (cherry picked from commit c7b4ed3)
Handle the combination of a write and a void return (cherry picked from commit 02cfb81)
Simplify and correct PermittedJumps computation (cherry picked from commit deefb01)
Improve insertion position of extracted methods (cherry picked from commit 40e4591)
Call getShorthandAssignmentValueSymbol rather than getSymbolAtLocation (cherry picked from commit aade971)
Only introduce return properties at the top level (cherry picked from commit 288a57c)
Forbid extraction of empty spans (cherry picked from commit be5c00f)
Re-enable extraction of a single token (cherry picked from commit 47b61ac)
selection Fixes microsoft#18546 (cherry picked from commit af49c60)
c354e51
to
27bede8
Compare
Built on top of #18516. If we don't merge that PR, this one will have to be redone since there are merge conflicts and one of these fixes is specific to type parameters.