forked from uutils/coreutils
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request uutils#5077 from starccy/ci-checkshell-workflow
ci: Add a new workflow to check and format shell scripts in `util` dir
- Loading branch information
Showing
1 changed file
with
74 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,74 @@ | ||
name: CheckScripts | ||
|
||
# spell-checker:ignore ludeeus mfinelli | ||
|
||
env: | ||
SCRIPT_DIR: 'util' | ||
|
||
on: | ||
push: | ||
branches: | ||
- main | ||
paths: | ||
- 'util/**/*.sh' | ||
pull_request: | ||
branches: | ||
- main | ||
paths: | ||
- 'util/**/*.sh' | ||
|
||
# End the current execution if there is a new changeset in the PR. | ||
concurrency: | ||
group: ${{ github.workflow }}-${{ github.ref }} | ||
cancel-in-progress: ${{ github.ref != 'refs/heads/main' }} | ||
|
||
jobs: | ||
shell_check: | ||
name: ShellScript/Check | ||
runs-on: ubuntu-latest | ||
permissions: | ||
contents: read | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: Run ShellCheck | ||
uses: ludeeus/action-shellcheck@master | ||
env: | ||
SHELLCHECK_OPTS: -s bash | ||
with: | ||
severity: warning | ||
scandir: ${{ env.SCRIPT_DIR }} | ||
format: tty | ||
|
||
shell_fmt: | ||
name: ShellScript/Format | ||
# no need to run in pr events | ||
# shfmt will be performed on main branch when the PR is merged | ||
if: github.event_name != 'pull_request' | ||
runs-on: ubuntu-latest | ||
needs: [ shell_check ] | ||
permissions: | ||
contents: write | ||
pull-requests: write | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: Setup shfmt | ||
uses: mfinelli/setup-shfmt@v2 | ||
- name: Run shfmt | ||
shell: bash | ||
run: | | ||
# show differs first for every files that need to be formatted | ||
# fmt options: bash syntax, 4 spaces indent, indent for switch-case | ||
echo "## show the differences between formatted and original scripts..." | ||
find ${{ env.SCRIPT_DIR }} -name "*.sh" -print0 | xargs -0 shfmt -ln=bash -i 4 -ci -d || true | ||
# perform a shell format | ||
echo "## perform a shell format..." | ||
# ignore the error code because `-d` will always return false when the file has difference | ||
find ${{ env.SCRIPT_DIR }} -name "*.sh" -print0 | xargs -0 shfmt -ln=bash -i 4 -ci -w | ||
- name: Commit any changes | ||
uses: EndBug/add-and-commit@v9 | ||
with: | ||
default_author: github_actions | ||
message: "style: auto format by CI (shfmt)" | ||
env: | ||
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
|