-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test installation in zsh and fish shells #139
Conversation
e2be8c0
to
2d942a8
Compare
.github/workflows/ci.yml
Outdated
run: sh -c '${{ matrix.shell }} -c "./install.sh"' | ||
- name: Install ${{matrix.shell}} using apt-get | ||
if: matrix.os == 'ubuntu-latest' && matrix.shell != 'sh' | ||
run: "sudo apt-get install -y ${{ matrix.shell }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you do something at the end to confirm installed?
.github/workflows/ci.yml
Outdated
|
||
test-install: | ||
strategy: | ||
fail-fast: false | ||
matrix: | ||
os: [ubuntu-latest, macos-latest, windows-latest] | ||
shell: [sh, bash, zsh] | ||
shell: [sh, bash, zsh, fish] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should not do these (just) by "shell" you should (also) do them also by "approach". In many cases the shell does not matter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
.github/workflows/ci.yml
Outdated
run: "brew update && brew install ${{ matrix.shell }}" | ||
|
||
- name: Update SHELL env variable | ||
run: "env SHELL=$(which ${{ matrix.shell }})" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we be using https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstepsshell instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i've tried this though have been running into syntax issues with shell: ${{ .. }}
approach. Hardcoding the shells worked
What was changed
Testing install .sh script in zsh and fish
Why?
testing zsh & fish, getting the green checkmark for CI
Checklist
Closes
How was this tested:
CI