forked from heavy-duty/rig
fix(ci): shellcheck -x, and lint commands/lib/ at all
The runner-* commands source commands/lib/runner-config.sh. Without -x, shellcheck will not follow that source even though the files carry a `source=SCRIPTDIR/...` directive, so it reported SC1091 on all three — and shellcheck exits non-zero on any finding, info-level included. commands/*.sh does not glob into lib/, so runner-config.sh was being linted by nothing at all. List it explicitly. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
parent
d4ab362964
commit
9c6c6c9477
1 changed files with 3 additions and 1 deletions
4
.github/workflows/ci.yml
vendored
4
.github/workflows/ci.yml
vendored
|
|
@ -9,6 +9,8 @@ jobs:
|
||||||
steps:
|
steps:
|
||||||
- uses: actions/checkout@v4
|
- uses: actions/checkout@v4
|
||||||
- name: shellcheck
|
- name: shellcheck
|
||||||
run: shellcheck install.sh bin/rig commands/*.sh test/cli.sh
|
# -x follows the `source=SCRIPTDIR/...` directives into commands/lib/.
|
||||||
|
# commands/*.sh does not glob into lib/, so list it explicitly.
|
||||||
|
run: shellcheck -x install.sh bin/rig commands/*.sh commands/lib/*.sh test/cli.sh
|
||||||
- name: cli tests
|
- name: cli tests
|
||||||
run: bash test/cli.sh
|
run: bash test/cli.sh
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue