Skip to content

Commit 949c5cb

Browse files
pieternfjakobs
andauthored
Bump bricks (#486)
This pulls in a change that modifies the args to `bricks sync SRC DST`. Co-authored-by: Fabian Jakobs <[email protected]>
1 parent ceb6324 commit 949c5cb

File tree

3 files changed

+5
-9
lines changed

3 files changed

+5
-9
lines changed

packages/databricks-vscode/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -622,7 +622,7 @@
622622
"scripts": {
623623
"vscode:prepublish": "rm -rf out && yarn run package:compile && yarn run package:copy-webview-toolkit",
624624
"package": "vsce package --baseContentUrl https://github.com/databricks/databricks-vscode/blob/${TAG:-main}/packages/databricks-vscode --baseImagesUrl https://raw.githubusercontent.com/databricks/databricks-vscode/${TAG:-main}/packages/databricks-vscode",
625-
"package:cli:fetch": "BRICKS_VERSION=0.0.22 && bash ./scripts/fetch-bricks-cli.sh ${BRICKS_VERSION} ${BRICKS_ARCH:-}",
625+
"package:cli:fetch": "BRICKS_VERSION=0.0.23 && bash ./scripts/fetch-bricks-cli.sh ${BRICKS_VERSION} ${BRICKS_ARCH:-}",
626626
"package:cli:link": "rm -f ./bin/bricks && ln -s ../../../../bricks/bricks bin",
627627
"package:compile": "yarn run esbuild:base",
628628
"package:copy-webview-toolkit": "cp ./node_modules/@vscode/webview-ui-toolkit/dist/toolkit.js ./out/toolkit.js",

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,13 @@ describe(__filename, () => {
4545
let {command, args} = cli.getSyncCommand(mapper, "incremental");
4646
assert.equal(
4747
[command, ...args].join(" "),
48-
"./bin/bricks sync --remote-path /Repos/[email protected]/notebook-best-practices --watch"
48+
"./bin/bricks sync . /Repos/[email protected]/notebook-best-practices --watch"
4949
);
5050

5151
({command, args} = cli.getSyncCommand(mapper, "full"));
5252
assert.equal(
5353
[command, ...args].join(" "),
54-
"./bin/bricks sync --remote-path /Repos/[email protected]/notebook-best-practices --watch --full"
54+
"./bin/bricks sync . /Repos/[email protected]/notebook-best-practices --watch --full"
5555
);
5656
});
5757

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

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,8 @@ export class CliWrapper {
5656
syncType: SyncType
5757
): Command {
5858
const command = this.extensionContext.asAbsolutePath("./bin/bricks");
59-
const args = [
60-
"sync",
61-
"--remote-path",
62-
syncDestination.remoteUri.path,
63-
"--watch",
64-
];
59+
const args = ["sync", ".", syncDestination.remoteUri.path, "--watch"];
60+
6561
if (syncType === "full") {
6662
args.push("--full");
6763
}

0 commit comments

Comments
 (0)