Skip to content

Move rename functionality to the language server - #2612

Merged
Mikhail Arkhipov (MikhailArkhipov) merged 79 commits into
microsoft:masterfrom
MikhailArkhipov:master
Sep 21, 2018
Merged

Mikhail Arkhipov (MikhailArkhipov) merged 79 commits into
microsoft:masterfrom
MikhailArkhipov:master

Conversation

@MikhailArkhipov

@MikhailArkhipov Mikhail Arkhipov (MikhailArkhipov) commented Sep 18, 2018 •

Copy link
Copy Markdown

Also fixes #2614

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please could you add a news entry file (new feature to add rename in language server). I would if i could, however you'd working off your master, and I can't apply changes directly in your master.

@MikhailArkhipov
Mikhail Arkhipov (MikhailArkhipov) merged commit 8b2a9b0 into microsoft:master Sep 21, 2018
Don Jayamanne (DonJayamanne) pushed a commit that referenced this pull request Oct 9, 2018
For #2690.

Similar to #2612. Note that `:`-based formatting isn't implemented in language server yet (microsoft/python-language-server#165), but it hasn't been enabled in the extension for anyone for a while now either (fixed in #2714/#2724).

Since line formatting is relatively new in language server, this likely shouldn't make it out into a release until the language server downloaded by default is new enough to include that functionality. I haven't been monitoring that side of things, so feel free to hold off on merging this until the right time.
@lock lock Bot locked as resolved and limited conversation to collaborators Jul 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LS download messages are poorly formatted

3 participants