Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Daniel Banck <dbanck@users.noreply.github.com>
  • Loading branch information
jpogran and dbanck authored Jan 26, 2024
1 parent 4bf409d commit ac5a596
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -768,7 +768,7 @@
},
{
"view": "terraform.providers",
"contents": "There are no installed providers found for the current open file.\n[Learn more about providers](https://www.terraform.io/docs/language/modules/develop/index.html)",
"contents": "There are no installed providers found for the current open file.\n[Learn more about providers](https://www.terraform.io/docs/language/providers/index.html)",
"when": "terraform.providers.noProviders === true"
},
{
Expand Down
2 changes: 1 addition & 1 deletion src/providers/moduleCalls.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ export class ModuleCallsDataProvider implements vscode.TreeDataProvider<ModuleCa
vscode.env.openExternal(vscode.Uri.parse(module.docsLink));
}
}),
vscode.window.onDidChangeActiveTextEditor(async (event: vscode.TextEditor | undefined) => {
vscode.window.onDidChangeActiveTextEditor(async () => {
// most of the time this is called when the user switches tabs or closes the file
// we already check for state inside the getModule function, so we can just call refresh here
this.refresh();
Expand Down
2 changes: 1 addition & 1 deletion src/providers/moduleProviders.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ export class ModuleProvidersDataProvider implements vscode.TreeDataProvider<Modu
constructor(ctx: vscode.ExtensionContext, private client: LanguageClient, private reporter: TelemetryReporter) {
ctx.subscriptions.push(
vscode.commands.registerCommand('terraform.providers.refreshList', () => this.refresh()),
vscode.window.onDidChangeActiveTextEditor(async (event: vscode.TextEditor | undefined) => {
vscode.window.onDidChangeActiveTextEditor(async () => {
// most of the time this is called when the user switches tabs or closes the file
// we already check for state inside the getprovider function, so we can just call refresh here
this.refresh();
Expand Down

0 comments on commit ac5a596

Please sign in to comment.