-
Notifications
You must be signed in to change notification settings - Fork 510
Feature request: "Go to Definition" works across all files in a workspace #499
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
Comments
Yep, we have a lot of work to do on the symbol features in PowerShell Editor Services. We need to start working on module-level analysis to enable this. |
This sounds close to #420. There should be some overlap but I'm not certain if/what unique changes there are to these two issues. |
Hi, any updates on this? I stumbled across this issue when figuring out if CodeLens was supposed to look for function names across multiple modules in the workspace. I think this is related? |
There's been some work to improve this. Probably not in the space you're hoping. I don't think I quite understand your comment - can you give an example, @rfoust? |
@TylerLeonhardt oops it's been a year, guess I'll reply now lol. :) I mean if I have multiple module folders open in the workspace and I'm looking at a function definition "function Get-Something" then right above it, it says "0 references" but there are really all kinds of references to it in other modules that I'm editing in my workspace, it just isn't seeing them. |
What's the scope of this issue? Be aware that the lookup is not module-aware, so conflicting function names might cause multiple returned definitions and wrong reference count. Tracking definitions across |
This issue has been marked as fixed. It has been automatically closed for housekeeping purposes. |
It'd be super-convenient for me if I could right-click on a function call click "Go to Definition" and it would find the module (if exists) that's in my current workspace, open up that file and then go to the function declaration.
The text was updated successfully, but these errors were encountered: