Go to Definition (F12) for import/using statements

[guidorice]

In julia-vscode extension, there seems to be no way to Go to Definition to see the toplevel source code of an imported package. I found I can do it from the REPL like this:

julia> import Package
julia> edit(Package)

It would be nice if just hitting F12 over an import or using statement in Julia code would accomplish the same thing as those 2 lines of code in the REPL.

Thanks so much for your efforts making Julia awesome in VScode. 👍🏽

[pfitzseb]

That already works, at least if the package is in the currently active environment.

[guidorice]

@pfitzseb, thanks for the prompt review of this issue. Respectfully, and admittedly I am very new to Julia, but also a longtime VSCode user with other languages, and I am having a hard time getting this to work, which is why i posted.
I did some more testing, and actually, it seems to work intermittently or only in certain circumstances.

Here is a workflow to reproduce. Can you identify anything I'm doing wrong? Probably something silly, or just not understanding how environments work in Julia.

$ git clone https://github.com/JuliaGeo/GeoJSON.jl.git
$ cd GeoJSON.jl/
$ code .
# Julia: Start REPL
# ]
# (@v1.7) pkg> activate .
  Activating project at `~/repos/jl/GeoJSON.jl`

(GeoJSON) pkg> instantiate
    Updating registry at `~/.julia/registries/General.toml`
    Updating `~/repos/jl/GeoJSON.jl/Project.toml`
  [cf35fbd7] + GeoInterface v0.5.7
  [0f8b85d8] + JSON3 v1.9.4
    Updating `~/repos/jl/GeoJSON.jl/Manifest.toml`
  [cf35fbd7] + GeoInterface v0.5.7
  [0f8b85d8] + JSON3 v1.9.4
  [69de0a69] + Parsers v2.3.0
  [3cdcf5f2] + RecipesBase v1.2.1
  [856f2bd8] + StructTypes v1.8.1
  [ade2ca70] + Dates
  [a63ad114] + Mmap
  [de0858da] + Printf
  [9a3f8284] + Random
  [ea8e919c] + SHA
  [9e88b42a] + Serialization
  [cf7118a7] + UUIDs
  [4ec0a83e] + Unicode
# Julia: Change Current Environment -> GeoJSON.jl
# Now put cursor over `import` or `using` statements, and hit F12
# See popup No definition found for 'Package', 
# e.g No definition found for 'GeoInterface'

[pfitzseb]

Ah, I see what's going on here. We're downloading cache files for packages (which contain all info we need for navigation, completion, docs, etc), but for GeoInterface specifically (well, probably for more packages) we can't automatically resolve and insert the package path in place of our placeholders.

I should have a fix for that in a bit.

[guidorice]

Cool, thanks much 👍🏽