Add a DefinitionProvider to support Go-To Definition #28

Activity

1. DanTup

   added this to the 0.6.0This milestone has been deleted milestone on Aug 6, 2016
2. DanTup

   added

   is enhancementAn enhancement or improvement that should be listed in release notes but is not a bug fix.An enhancement or improvement that should be listed in release notes but is not a bug fix.

   on Aug 6, 2016
3. DanTup

   changed the title [-]add a DefinitionProvider[/-] [+]Add a DefinitionProvider to support Go-To Definition[/+] on Aug 6, 2016
4. DanTup commented on Aug 6, 2016

   

   DanTup

   on Aug 6, 2016

   Last edited by DanTup

   Member

   More actions

   VSCode doc
   Aalysis Server Doc
5. DanTup

   closed this as completedin 44fdf3con Aug 6, 2016
6. DanTup commented on Aug 6, 2016

   

   DanTup

   on Aug 6, 2016

   Member

   More actions

   This is done, but I can't make it work for SDK files so I've raised #35 to add that once I figured it out (the problem seems to be no startLine/startColumn for SDK files).
7. devoncarew commented on Aug 6, 2016

   

   devoncarew

   on Aug 6, 2016

   Last edited by devoncarew

   ContributorAuthor

   More actions

   IIRC the data for this is sent via notifications (analysis.navigation) so I'll need to keep hold of all of that data somewhere for when it's needed?

   Navigation can come in as events, or as an explicit call, but looks like you found analysis.getNavigation already :)
8. devoncarew commented on Aug 6, 2016

   

   devoncarew

   on Aug 6, 2016

   ContributorAuthor

   More actions

   Re: that bug, that's probably due to some summary work - the SDK analysis is summarized and stored on disk, instead of being analyzed for each project.
9. DanTup commented on Aug 6, 2016

   

   DanTup

   on Aug 6, 2016

   Member

   More actions

   Ok, I've just filtered them out for now and I'll deal with it in #35 if/when there's a good solution.