rewrite multihop GTD with single jedi API call once jedi supports it #446

Activity

1. tkrabel-db commented on Oct 12, 2023

   

   tkrabel-db

   on Oct 12, 2023

   Contributor

   More actions

   @ccordoba12 @krassowski can you please close this?

   In general, we should keep the issue section clean. Happy to do it if you give me the according rights! :)
2. krassowski commented on Oct 12, 2023

   

   krassowski

   on Oct 12, 2023

   Last edited by krassowski

   Member

   More actions

   This issue was opened because after discussion with the Jedi author the conclusion was that:

   1. The workaround proposed in Allow Jedi "goto" to perform multiple hops for "go to definition" #443 is not ideal (among others performance-wise)
   2. The change should happen in Jedi itself

   As the philosophy of pylsp is to upstream as much as possible, I believe this should be ultimately implemented in Jedi and then the workaround that was merged should be removed. This issue tracks removal of the workaround. As such I do not see a reason to close it.

   👍React with 👍1Reacted by Carlos Cordoba
3. tkrabel-db commented on Oct 12, 2023

   

   tkrabel-db

   on Oct 12, 2023

   Contributor

   More actions

   Ah, thanks for the clarification. My bad!