Skip to content

Commit 136cb1c

Browse files
hanwen-clusterhanwen-pcluste
authored andcommitted
Add Shellcheck to GitHub Action
Shellcheck is a static analyzer for Shell script. This commit only enables the error checks, and disables the info and warning checks. Because there would've been a lot to fix if WARNING/INFO is enabled. This commit fixes an error detected by Shellcheck Signed-off-by: Hanwen <[email protected]>
1 parent 9a3d069 commit 136cb1c

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

.github/workflows/ci.yml

+10
Original file line numberDiff line numberDiff line change
@@ -79,3 +79,13 @@ jobs:
7979
run: pip install tox
8080
- name: Run Tox
8181
run: tox -e ${{ matrix.toxenv }}
82+
shellcheck:
83+
name: Shellcheck
84+
runs-on: ubuntu-latest
85+
steps:
86+
- uses: actions/checkout@v2
87+
- name: Run ShellCheck
88+
uses: ludeeus/action-shellcheck@master
89+
with:
90+
severity: error
91+
ignore_paths: third-party

system_tests/install_cinc.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ elif [ `echo "${RELEASE}" | grep '^ubuntu\.20'` ]; then
2222
OS='ubuntu2004'
2323
else
2424
echo "Operating System '${RELEASE}' is not supported. Failing build."
25-
exit -1
25+
exit 1
2626
fi
2727

2828
if [ `echo "${OS}" | grep -E '^(alinux|centos)'` ]; then

0 commit comments

Comments
 (0)