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

Implement refactor: Inline local variable #545

Closed
DanTup opened this issue Feb 8, 2018 · 5 comments
Closed

Implement refactor: Inline local variable #545

DanTup opened this issue Feb 8, 2018 · 5 comments
Labels
in editor Relates to code editing or language features in lsp/analysis server Something to be fixed in the Dart analysis server is enhancement
Milestone

Comments

@DanTup
Copy link
Member

DanTup commented Feb 8, 2018

No description provided.

@DanTup DanTup added is enhancement in editor Relates to code editing or language features labels Feb 8, 2018
@DanTup DanTup added this to the On Deck milestone Feb 8, 2018
@DanTup DanTup added the blocked on vs code / lsp / dap Requires a change in VS Code to progress label Feb 12, 2018
@DanTup
Copy link
Member Author

DanTup commented 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 DanTup removed the blocked on vs code / lsp / dap Requires a change in VS Code to progress label Feb 21, 2018
@DanTup
Copy link
Member Author

DanTup commented Dec 19, 2018

Requires dart-lang/sdk#35448.

@DanTup DanTup added the blocked on dart / flutter Requires a change in Dart or Flutter to progress label Dec 19, 2018
@DanTup DanTup modified the milestones: On Deck, Backlog Mar 19, 2019
@DanTup DanTup added the in lsp/analysis server Something to be fixed in the Dart analysis server label Jun 1, 2020
@DanTup DanTup modified the milestones: Backlog, On Deck Nov 18, 2020
@DanTup DanTup removed the blocked on dart / flutter Requires a change in Dart or Flutter to progress label Nov 18, 2020
@svarlet
Copy link

svarlet commented Mar 4, 2021

Hi @DanTup , what's the status on this one?

@DanTup DanTup modified the milestones: On Deck, v3.21.0 Mar 8, 2021
@DanTup
Copy link
Member Author

DanTup commented Mar 8, 2021

No progress, although I wonder if it might no longer be blocked by the Dart issue above for LSP. I'll assign to this milestone to have a look to see if that's the case (though no guarantees).

@DanTup DanTup modified the milestones: v3.21.0, v3.22.0 Mar 31, 2021
dart-bot pushed a commit to dart-lang/sdk that referenced this issue Apr 27, 2021
Fixes Dart-Code/Dart-Code#545.

Change-Id: Ibee992ab1e5ff13b849d6e1151e21d9229087d73
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/197046
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
@DanTup
Copy link
Member Author

DanTup commented Apr 27, 2021

Done in dart-lang/sdk@8ae2eb9.

@DanTup DanTup closed this as completed Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in editor Relates to code editing or language features in lsp/analysis server Something to be fixed in the Dart analysis server is enhancement
Projects
None yet
Development

No branches or pull requests

2 participants