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

Check for submodules when fetching schemas from the registry #1772

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

dbanck
Copy link
Member

@dbanck dbanck commented Jul 19, 2024

Closes #1771
Closes hashicorp/vscode-terraform#1800


The registry client always used the Inputs and Outputs from the root module when fetching and storing a schema for a module with a registry address.
We now check if the source address contains a sub directory and use the Inputs and Outputs from the matching submodule.

UX Before

CleanShot 2024-07-19 at 17 26 25@2x

UX After

CleanShot 2024-07-19 at 17 26 48@2x

@dbanck dbanck added the bug Something isn't working label Jul 19, 2024
The registry client always used the Inputs and Outputs from the root
module when fetching and storing a schema for a module with a registry
address.
We now check if the source address contains a sub directory and use
the Inputs and Outputs from the matching submodule.
@dbanck dbanck marked this pull request as ready for review July 19, 2024 15:28
@dbanck dbanck requested a review from a team as a code owner July 19, 2024 15:28
@dbanck dbanck merged commit 7d23d0f into main Jul 22, 2024
23 checks passed
@dbanck dbanck deleted the b-store-submodule-schema branch July 22, 2024 08:40
Copy link

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.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
2 participants