Skip to content

Commit e9b9a05

Browse files
fix: swapped output of symlink (#3361)
## Changes Bug fix: before, on installing pipelines for the first time, users are told that pipelines is reinstalled, while when overwriting their pipelines symlink, users are told pipelines is installed, without mention of the deletion. Swapped output of print statements to match the actual expected output. --------- Co-authored-by: Jeffery Cheng <105739315+jefferycheng1@users.noreply.github.com>
1 parent aa80a4d commit e9b9a05

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

acceptance/pipelines/install-pipelines-cli/output.txt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11

22
=== install pipelines cli
33
>>> [CLI] install-pipelines-cli -d ./subdir
4-
found existing pipelines installation at subdir/pipelines. Pipelines is successfully reinstalled in directory "./subdir"
4+
pipelines successfully installed in directory "./subdir"
55

66
>>> ./subdir/pipelines
77
Pipelines CLI
@@ -46,7 +46,7 @@ Exit code: 1
4646

4747
=== databricks executable called with alias
4848
>>> ./subdir/notdatabricks install-pipelines-cli -d ./subdir
49-
found existing pipelines installation at subdir/pipelines. Pipelines is successfully reinstalled in directory "./subdir"
49+
pipelines successfully installed in directory "./subdir"
5050

5151
>>> ./subdir/pipelines
5252
Pipelines CLI
@@ -80,10 +80,10 @@ Use "pipelines [command] --help" for more information about a command.
8080

8181
=== overwrites broken symlink pointing to databricks
8282
>>> ./subdir/olddatabricks install-pipelines-cli -d ./subdir
83-
found existing pipelines installation at subdir/pipelines. Pipelines is successfully reinstalled in directory "./subdir"
83+
pipelines successfully installed in directory "./subdir"
8484

8585
>>> [CLI] install-pipelines-cli -d ./subdir
86-
pipelines successfully installed in directory "./subdir"
86+
found existing pipelines installation at subdir/pipelines. Pipelines CLI is successfully reinstalled in directory "./subdir"
8787

8888
>>> ./subdir/pipelines
8989
Pipelines CLI

cmd/pipelines/install_pipelines_cli.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ func installPipelinesSymlink(ctx context.Context, directory string) error {
3838
if err != nil {
3939
return err
4040
}
41-
cmdio.LogString(ctx, fmt.Sprintf("found existing pipelines installation at %s. Pipelines is successfully reinstalled in directory %q", pipelinesPath, dir))
41+
cmdio.LogString(ctx, fmt.Sprintf("pipelines successfully installed in directory %q", dir))
4242
return nil
4343
}
4444

@@ -54,7 +54,7 @@ func installPipelinesSymlink(ctx context.Context, directory string) error {
5454
if err != nil {
5555
return err
5656
}
57-
cmdio.LogString(ctx, fmt.Sprintf("pipelines successfully installed in directory %q", dir))
57+
cmdio.LogString(ctx, fmt.Sprintf("found existing pipelines installation at %s. Pipelines CLI is successfully reinstalled in directory %q", pipelinesPath, dir))
5858
return nil
5959
}
6060
return err

0 commit comments

Comments
 (0)