You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't think putting refactors on the lightbulb is a good idea, as they show up so frequently that there will just be bulbs everywhere always. I suggest these go on the context menu:
Hi @DanTup !
I just noticed the completion of the similar issue about inlining variables. What's the status on this issue? Inlining methods is an even more useful refactoring 😍
Activity
DanTup commentedon Feb 12, 2018
I don't think putting refactors on the lightbulb is a good idea, as they show up so frequently that there will just be bulbs everywhere always. I suggest these go on the context menu:
microsoft/vscode#9827
DanTup commentedon Dec 19, 2018
Requires dart-lang/sdk#35448.
svarlet commentedon Apr 28, 2021
Hi @DanTup !
I just noticed the completion of the similar issue about inlining variables. What's the status on this issue? Inlining methods is an even more useful refactoring 😍
DanTup commentedon Apr 28, 2021
@svarlet hoping to look at this shortly too :-)
DanTup commentedon Apr 29, 2021
I've got a change open for this at https://dart-review.googlesource.com/c/sdk/+/197380/.
[analyzer] Support "Inline Method" refactor in LSP
DanTup commentedon Apr 29, 2021
Fixed in dart-lang/sdk@a89fccb.