Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Go to Definition" does not work with umbrella apps #514

Closed
Nezteb opened this issue Mar 16, 2021 · 1 comment
Closed

"Go to Definition" does not work with umbrella apps #514

Nezteb opened this issue Mar 16, 2021 · 1 comment

Comments

@Nezteb
Copy link

Nezteb commented Mar 16, 2021

Environment

In this example, I'm trying to "Go to Definition" on Phoenix's render function, which is being used within an umbrella app:
Screen Shot 2021-03-15 at 9 18 00 PM

I'd expect it to bring me to the deps/phoenix folder like so:
Screen Shot 2021-03-15 at 9 18 52 PM

In 2017, @JakeBecker said that umbrella apps are not supported: JakeBecker/vscode-elixir-ls#6 (comment)

I've searched this repo as well as the VS Code extension repo, and umbrella app support doesn't seem to be listed anywhere on any roadmap. The lack of support for umbrella apps also isn't listed in either project's README. There are some old issues for both projects that loosely touch on this subject, but they never explicitly call out the lack of "Go to Definition" support, so I figured I'd create this one until someone can decide what the best thing to do is.

@axelson
Copy link
Member

axelson commented Mar 18, 2021

Hi 👋!, thanks for the report! Umbrella applications are expected to be fully supported. It looks like you are running into #193 since that render function is actually a phoenix function imported by the use statement: https://hexdocs.pm/phoenix/Phoenix.Controller.html?#render/3

I'm going to close this for now, but if you find other issues specific to umbrellas, then please file a new issue.

@axelson axelson closed this as completed Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants