Skip to content
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

Run shellcheck on swiftly-install.sh #64

Open
patrickfreed opened this issue Sep 21, 2023 · 3 comments
Open

Run shellcheck on swiftly-install.sh #64

patrickfreed opened this issue Sep 21, 2023 · 3 comments
Labels
swiftly-install Issue related to the installation script

Comments

@patrickfreed
Copy link
Collaborator

shellcheck is a static analysis tool for bash which can catch a number of common bash issues. We should run it on swiftly-install.sh and include it in the soundness check CI task.

@patrickfreed patrickfreed added the swiftly-install Issue related to the installation script label Sep 21, 2023
@adam-fowler
Copy link
Member

This should be part of the CI setup as well

@Zingam
Copy link

Zingam commented Dec 19, 2023

bash is a kind of deprecated on macOS. Better use posix shell for max compatibility.

@adam-fowler
Copy link
Member

bash is a kind of deprecated on macOS. Better use posix shell for max compatibility.

Do you mean deprecated in that it isn't the default shell on macOS anymore or that Apple intend to delete it in a future release of macOS. If it is the second can you post where Apple announced this.

Currently bash is available on pretty much every Linux release. I'll have to let @patrickfreed cover any issues with falling back to posix shell.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
swiftly-install Issue related to the installation script
Projects
None yet
Development

No branches or pull requests

3 participants