Skip to content

Commit 548717e

Browse files
committed
run shellcheck and incorporate shellcheck in gh actions
1 parent bb2de17 commit 548717e

File tree

2 files changed

+11
-2
lines changed

2 files changed

+11
-2
lines changed

.github/workflows/docker-tests.yml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,13 +10,22 @@ concurrency:
1010
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }}
1111

1212
jobs:
13+
shellcheck:
14+
name: Shellcheck
15+
runs-on: ubuntu-latest
16+
17+
steps:
18+
- name: Run ShellCheck
19+
- uses: actions/checkout@v4
20+
run: find . -type f -name "*.sh" -exec shellcheck {} +
21+
1322
tests:
1423
name: Validate Platform Installation
1524
runs-on: ubuntu-latest
1625

1726
steps:
1827
- name: Checkout
19-
uses: actions/checkout@v3
28+
uses: actions/checkout@v4
2029

2130
- name: Install Platform
2231
run: |

install-and-launch.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,7 @@ install_sublime() {
444444

445445
if [ "$clone_platform" = "true" ]; then
446446
print_info "Cloning Sublime Platform repo..."
447-
if ! git clone -b $remote_branch --depth=1 https://github.com/sublime-security/sublime-platform.git; then
447+
if ! git clone -b "$remote_branch" --depth=1 https://github.com/sublime-security/sublime-platform.git; then
448448
print_error "Failed to clone Sublime Platform repo\n"
449449
printf "Troubleshooting tips: https://docs.sublimesecurity.com/docs/quickstart-docker#troubleshooting\n\n"
450450
printf "You may need to run the following command before retrying installation:\n\n"

0 commit comments

Comments
 (0)