Skip to content

Commit

Permalink
Improve comment on why we add ~/bin to PATH clearer in the test workflow
Browse files Browse the repository at this point in the history
  • Loading branch information
shreyas-goenka committed Feb 26, 2024
1 parent 76b6d51 commit 1aa0ec9
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 1aa0ec9

Please sign in to comment.