Skip to content

Commit

Permalink
🐛 do not need to dupe context (#18)
Browse files Browse the repository at this point in the history
Signed-off-by: David Zager <dzager@redhat.com>
  • Loading branch information
djzager authored Sep 25, 2024
1 parent 8c76e8d commit f8eb03a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion vscode/src/VsCodeExtension.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,6 @@ export class VsCodeExtension {
});

// Commands
registerAllCommands(context, this.extensionContext, this.sidebar);
registerAllCommands(context, this.sidebar);
}
}
5 changes: 2 additions & 3 deletions vscode/src/commands.ts
Original file line number Diff line number Diff line change
Expand Up @@ -83,11 +83,10 @@ const commandsMap: (

export function registerAllCommands(
context: vscode.ExtensionContext,
extensionContext: vscode.ExtensionContext,
sidebar: KonveyorGUIWebviewViewProvider,
sidebar: KonveyorGUIWebviewViewProvider
) {
for (const [command, callback] of Object.entries(
commandsMap(extensionContext, sidebar),
commandsMap(context, sidebar)
)) {
context.subscriptions.push(
vscode.commands.registerCommand(command, callback),
Expand Down

0 comments on commit f8eb03a

Please sign in to comment.