Skip to content

Commit

Permalink
update syntax check to shared action (#15)
Browse files Browse the repository at this point in the history
This pull request includes a change to the workflow file
`.github/workflows/lint-shell.yml` to improve the naming and execution
of a job. The `sh-noexec-verify` job was renamed to
`simple-shell-syntax-check` and the `run` command was replaced with the
`uses` command to run the `Klintrup/simple-shell-syntax-check@v1`
action.

Main workflow change:

* <a
href="diffhunk://#diff-cb01ce593de6f1d1a9c247dde8298db8c7210be7d0654e00597dd13c8e0e565eL10-R14">`.github/workflows/lint-shell.yml`</a>:
Renamed the `sh-noexec-verify` job to `simple-shell-syntax-check` and
replaced the `run` command with the `uses` command to run the
`Klintrup/simple-shell-syntax-check@v1` action.
  • Loading branch information
Klintrup authored Dec 3, 2023
1 parent 2959ba8 commit 4bd1d70
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/lint-shell.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,11 @@ permissions:

jobs:
sh:
name: sh-noexec-verify
name: simple-shell-syntax-check
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Run ShellCheck
run: sh -n check_smartarray.sh
- uses: Klintrup/simple-shell-syntax-check@v1
shellcheck:
name: shellcheck
runs-on: ubuntu-latest
Expand Down

0 comments on commit 4bd1d70

Please sign in to comment.