Skip to content

Commit e1d8abf

Browse files
Use spawn options from lazy terminal as is (#243)
1 parent c2ce32b commit e1d8abf

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

packages/databricks-vscode/src/cli/BricksTasks.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,10 +54,10 @@ describe(__filename, () => {
5454
);
5555

5656
assert.deepEqual(terminal.getProcessOptions(), {
57-
cwd: "/path/to/local/workspace",
57+
cwd: Uri.file("/path/to/local/workspace").fsPath,
5858
env: {
5959
/* eslint-disable @typescript-eslint/naming-convention */
60-
BRICKS_ROOT: "/path/to/local/workspace",
60+
BRICKS_ROOT: Uri.file("/path/to/local/workspace").fsPath,
6161
DATABRICKS_CONFIG_PROFILE: "profile",
6262
HOME: process.env.HOME,
6363
PATH: process.env.PATH,

packages/databricks-vscode/src/cli/BricksTasks.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,8 +95,7 @@ class CustomSyncTerminal implements Pseudoterminal {
9595

9696
private startSyncProcess() {
9797
this.syncProcess = spawn(this.cmd, this.args, {
98-
env: this.options?.env,
99-
cwd: this.options?.cwd,
98+
...this.options,
10099
});
101100

102101
// Log the sync command being run, its args and any env overrides done by

0 commit comments

Comments
 (0)