Skip to content
This repository has been archived by the owner on Apr 14, 2022. It is now read-only.

Go to definition does not work in new LS #850

Closed
DonJayamanne opened this issue Apr 1, 2019 · 5 comments
Closed

Go to definition does not work in new LS #850

DonJayamanne opened this issue Apr 1, 2019 · 5 comments
Assignees

Comments

@DonJayamanne
Copy link

@DonJayamanne commented on Sun Mar 31 2019

Works with Jedi but not LS

A picture says a thousand words...

  • Mac
  • Python 3.7
  • Use behave repo
    gotodef
@MikhailArkhipov
Copy link

@DonJayamanne - still broken? There were a number of fixes since.

@amittleider
Copy link

I experience this issue often in my code base. I also experience a different, but related issue where "Go to definition" will incorrectly move me to the import statement at the top of the file, instead of actually jumping to the definition. Here's a gif:

ms_language_server_go_to_def

@DonJayamanne
Copy link
Author

@MikhailArkhipov

It works for me now, apologies for the delay.

@MikhailArkhipov
Copy link

@amittleider - this is by design. First navigation takes you where variable is first defined in the current file, then navigation from import takes you to the library. If you prefer different behavior, please file a separate issue. I guess it would be a setting controlling if the navigation should be direct.

@amittleider
Copy link

Hi @MikhailArkhipov , thanks for the reply. I have opened a new issue here: #1138 .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants