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 support for duplicate module names. #48

Open
haydnevans1 opened this issue May 16, 2024 · 2 comments
Open

Go to Definition support for duplicate module names. #48

haydnevans1 opened this issue May 16, 2024 · 2 comments

Comments

@haydnevans1
Copy link

Hi, really love the extension! I work in a codebase where we often have multiple modules of the same name. The current functionality of the extension seems to just pick the first module it finds of that name. Is it possible to get a list of modules that match the name when using go to Definition?

Thanks.

@kkanhere
Copy link
Collaborator

Yep. The language server picks the first match. Since the protocol does support providing multiple targets, I believe this can be supported. Added todo list. Also note that pull requests are always welcome.

@haydnevans1
Copy link
Author

Ok great! Good to know it is very possible, I may make a pull request and give it a go over the weekend.

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