From 1aa0ec999f425202afe6685d7ac659a1ea1b6d0d Mon Sep 17 00:00:00 2001 From: Shreyas Goenka Date: Mon, 26 Feb 2024 14:40:20 +0100 Subject: [PATCH] Improve comment on why we add ~/bin to PATH clearer in the test workflow --- .github/workflows/test.yml | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 0b51f3d..2806e14 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -112,8 +112,9 @@ jobs: with: path: ./setup-cli - # Append ~/bin to the PATH. This helps with the assertions in the next steps. - - run: echo "PATH=$PATH:$HOME/bin" >> $GITHUB_ENV + # When run on DBR, the installation scripts installs the CLI to ~/bin. Add + # it to $PATH to make assertions work. + - run: echo "$HOME/bin" >> $GITHUB_PATH - name: Assert databricks CLI is not already installed run: ./setup-cli/assert/not-installed.sh @@ -185,8 +186,9 @@ jobs: with: path: ./setup-cli - # Append ~/bin to the PATH. This helps with the assertions in the next steps. - - run: echo "PATH=$PATH:$HOME/bin" >> $GITHUB_ENV + # When run on DBR, the installation scripts installs the CLI to ~/bin. Add + # it to $PATH to make assertions work. + - run: echo "$HOME/bin" >> $GITHUB_PATH - name: Assert databricks CLI is not already installed run: ./setup-cli/assert/not-installed.sh