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

Subscribe to updates of module.providers data #902

Merged
merged 1 commit into from
May 3, 2022

Conversation

jpogran
Copy link
Contributor

@jpogran jpogran commented Apr 29, 2022

Adds support for a client to register a command that will refresh clide side data based on changes the server has detected in the workspace.

This specifically adds a new command to refresh the module providers shown client side. The existing Module hook system is used to detect changes in the workspace and notify the client that data has changed.

@jpogran jpogran self-assigned this Apr 29, 2022
@jpogran jpogran added the enhancement New feature or request label Apr 29, 2022
@jpogran jpogran added this to the v0.28.0 milestone Apr 29, 2022
@jpogran jpogran linked an issue Apr 29, 2022 that may be closed by this pull request
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request Apr 29, 2022
This commit automatically refreshes the Terraform Providers View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleProviders` request which terraform-ls can call to refresh the data shown in the Terraform Providers view. The terraform-ls server was modified to send `client.refreshModuleProviders` requests in hashicorp/terraform-ls#902
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request Apr 29, 2022
This commit automatically refreshes the Terraform Providers View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleProviders` request which terraform-ls can call to refresh the data shown in the Terraform Providers view. The terraform-ls server was modified to send `client.refreshModuleProviders` requests in hashicorp/terraform-ls#902
@jpogran jpogran force-pushed the gh-723-module-providers-sub branch 2 times, most recently from 67bb8b8 to e3cc8a8 Compare April 29, 2022 14:00
Adds support for a client to register a command that will refresh clide side data based on changes the server has detected in the workspace.

This specifically adds a new command to refresh the module providers shown client side. The existing Module hook system is used to detect changes in the workspace and notify the client that data has changed.
@jpogran jpogran force-pushed the gh-723-module-providers-sub branch from e3cc8a8 to a7bf07e Compare April 29, 2022 14:07
@jpogran jpogran marked this pull request as ready for review April 29, 2022 14:07
@jpogran jpogran requested a review from a team as a code owner April 29, 2022 14:07
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request Apr 29, 2022
This commit automatically refreshes the Terraform Module Calls View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleCalls` request which terraform-ls can call to refresh the data shown in the Terraform Module Calls view. The terraform-ls server was modified to send `client.refreshModuleCalls` requests in hashicorp/terraform-ls#902
@jpogran jpogran marked this pull request as draft April 29, 2022 16:41
@jpogran jpogran force-pushed the gh-723-module-providers-sub branch from db03c5b to a7bf07e Compare April 29, 2022 16:54
@jpogran jpogran marked this pull request as ready for review May 2, 2022 15:59
Copy link
Member

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :shipit:

I found what I believe is unrelated bug while testing this, filed under #906 - we can take a look in a separate PR later.

@jpogran jpogran merged commit 0f5e46e into main May 3, 2022
@jpogran jpogran deleted the gh-723-module-providers-sub branch May 3, 2022 13:24
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request May 3, 2022
…in open document (#1084)

This commit automatically refreshes the Terraform Providers View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleProviders` request which terraform-ls can call to refresh the data shown in the Terraform Providers view. The terraform-ls server was modified to send `client.refreshModuleProviders` requests in hashicorp/terraform-ls#902
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request May 4, 2022
This commit automatically refreshes the Terraform Module Calls View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleCalls` request which terraform-ls can call to refresh the data shown in the Terraform Module Calls view. The terraform-ls server was modified to send `client.refreshModuleCalls` requests in hashicorp/terraform-ls#902
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request May 4, 2022
This commit automatically refreshes the Terraform Module Calls View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleCalls` request which terraform-ls can call to refresh the data shown in the Terraform Module Calls view. The terraform-ls server was modified to send `client.refreshModuleCalls` requests in hashicorp/terraform-ls#902
jpogran added a commit to hashicorp/vscode-terraform that referenced this pull request May 9, 2022
This commit automatically refreshes the Terraform Module Calls View when providers change in open document.

This introduces a StaticFeature which registers a handler for the `client.refreshModuleCalls` request which terraform-ls can call to refresh the data shown in the Terraform Module Calls view. The terraform-ls server was modified to send `client.refreshModuleCalls` requests in hashicorp/terraform-ls#902
@github-actions
Copy link

github-actions bot commented Jun 3, 2022

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 Jun 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow subscribing to updates of module.providers data
2 participants