python-rope / pylsp-rope

Extended refactoring capabilities for python-lsp-server using Rope
MIT License
118 stars 5 forks source link

[Question] Need to add "typing_extensions" to dependencies? #6

Closed yaegassy closed 3 years ago

yaegassy commented 3 years ago

Description

Hi, typing_extensions is required for "pylsp-rope" to work, right? (Checking the code of "pylsp-rope", it seems that typing_extensions is used)

Normally, typing_extensions is not included when python-lsp-server is installed.

DEMO (mp4)

https://user-images.githubusercontent.com/188642/136782751-77aebb1c-11a1-451e-b1ee-d5663560e6cf.mp4

Note

If you install pylsp-mypy, typing_extensions will be installed since mypy is present

If you add typing_extensions to your pylsp-rope dependencies, you may need to be careful not to conflict with the pylsp-mypy version.

Misc

lieryan commented 3 years ago

Hi @yaegassy, thank you for using pylsp-rope and thank you for writing up this detailed report.

I've updated and released pylsp-rope==0.1.6 to require typing-extensions as a runtime dependency.