-
Notifications
You must be signed in to change notification settings - Fork 132
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
Add version to module calls #874
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code LGTM, but I think we should create an issue upstream in https://github.com/microsoft/vscode to clarify the behaviour when both link & go-to-definition is associated with the same range.
I can test & re-review once the upstream PRs are merged and this is rebased.
ec06d5e
to
a69ec20
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Depends on
This PR populates the version field for
ModuleCall
and pulls in the latestterraform-schema
andhcl-lang
changes.All this enables linking the source attribute for remote modules to the documentation.