mirror of https://github.com/helm/helm
This commit adds a new GitHub Actions workflow that runs ShellCheck on all bash scripts in the repository. ShellCheck is a static analysis tool for shell scripts that detects bugs, security issues, and style problems. Changes: - Add .github/workflows/shellcheck.yml workflow that runs on push and pull requests - Fix bash script issues in scripts/release-notes.sh: - Quote all variable expansions to prevent word splitting (SC2086) - Use proper error checking instead of checking $? (SC2181) The workflow runs ShellCheck on all .sh files except testdata and generated directories. Signed-off-by: Adesh Deshmukh <adeshkd123@gmail.com>pull/31969/head
Adesh Deshmukh
4 months ago
parent
b3927b3900
commit
3b47bb2e34
@ -0,0 +1,37 @@
|
||||
name: ShellCheck
|
||||
|
||||
on:
|
||||
push:
|
||||
branches:
|
||||
- "main"
|
||||
- "dev-v3"
|
||||
- "release-**"
|
||||
pull_request:
|
||||
branches:
|
||||
- "main"
|
||||
- "dev-v3"
|
||||
|
||||
permissions:
|
||||
contents: read
|
||||
|
||||
jobs:
|
||||
shellcheck:
|
||||
name: shellcheck
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- name: Checkout
|
||||
uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # pin@v6.0.2
|
||||
|
||||
- name: Run shellcheck
|
||||
run: |
|
||||
sudo apt-get update
|
||||
sudo apt-get install -y shellcheck
|
||||
find . -type f -name "*.sh" \
|
||||
! -path "./internal/chart/v3/loader/testdata/*" \
|
||||
! -path "./internal/chart/v2/loader/testdata/*" \
|
||||
! -path "./pkg/chart/v2/loader/testdata/*" \
|
||||
! -path "./pkg/cmd/testdata/*" \
|
||||
! -path "./pkg/postrenderer/testdata/*" \
|
||||
! -path "./.git/*" \
|
||||
! -path "./node_modules/*" \
|
||||
-exec shellcheck {} +
|
||||
Loading…
Reference in new issue