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

Rework rename/move refactoring support #1815

Merged
merged 2 commits into from
Mar 24, 2021

Conversation

testforstephen
Copy link
Collaborator

Signed-off-by: Jinbo Wang [email protected]

It requires eclipse-jdtls/eclipse.jdt.ls#1445

  • Should use onWillRenameFiles listener to handle the rename refactoring because it supports returning an edit via e.waitUntil API. Also another benefit is that it will composite the edit with the undo stack of the file operation, that means clicking ctrl + z once will undo the file rename operation and the corresponding refactoring together.
  • Remove the old preview feature. We're going to delegate the preview capability to the client itself, no need to handle it in the extension side any more.

@testforstephen
Copy link
Collaborator Author

it's more stable in 1.53.

rename-refactoring.mov

@testforstephen testforstephen linked an issue Feb 24, 2021 that may be closed by this pull request
testforstephen referenced this pull request Feb 24, 2021
- VSCode 1.53 bumped minimum GLIBCXX to 3.4.21 making it difficult to
  run tests on older platforms. Instead of disabling, we can use 1.52.1
  runtime temporarily and eventually upgrade infrastructure.
- https://code.visualstudio.com/updates/v1_53#_electron-11-update

Signed-off-by: Roland Grunberg <[email protected]>
@testforstephen testforstephen merged commit 2c97a64 into redhat-developer:master Mar 24, 2021
@testforstephen testforstephen deleted the jinbo_move branch March 24, 2021 02:52
@testforstephen testforstephen added this to the End March 2021 milestone Mar 24, 2021
@testforstephen testforstephen removed this from the End March 2021 milestone Apr 6, 2021
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.

Refactor when moving files
3 participants