Skip to content

ci: add GitHub Actions workflow for shellcheck + bash -n + smoke tests#22

Closed
pstuart wants to merge 1 commit into
mainfrom
fix/auto-scan-20260505-ci
Closed

ci: add GitHub Actions workflow for shellcheck + bash -n + smoke tests#22
pstuart wants to merge 1 commit into
mainfrom
fix/auto-scan-20260505-ci

Conversation

@pstuart

@pstuart pstuart commented May 5, 2026

Copy link
Copy Markdown
Owner

Summary

Auto-scan 2026-05-05 deep-dispatch pass on Barista. Adds first GitHub Actions CI workflow per fleet pattern (10th this session: better-budget #41, stock-trainer #35, Better-legacy #36, bookmarks #27, save-for-later #38, flyck #61, better-bible-trivia #34, better-pastoral-care #27, tennis-story #76, this one).

Changes Made

ci: add GitHub Actions workflow for shellcheck + bash -n + smoke tests

  • What: added .github/workflows/ci.yml.
  • Why: no CI existed; every change relied on local runs.
  • How: ubuntu-latest runner, runs:
    • bash -n syntax check on every .sh file
    • shellcheck informationally (NOT a hard gate — see "What Was NOT Changed")
    • tests/test_sandbox.sh smoke suite
  • Logs shellcheck --version + bash --version so baseline drifts are visible going forward — issue tooling: ShellCheck baseline drift — 234 findings vs dossier's 50; propose .shellcheckrc to stabilize #21 was caused by an invisible ShellCheck 0.10.x → 0.11.0 bump that 7× SC2155 noise.

What Was NOT Changed

Verification

  • bash -n barista.sh install.sh modules/*.sh tests/*.sh: PASS
  • bash tests/test_sandbox.sh: 4/4 PASS
  • Workflow YAML lints with actionlint (passing — verified locally; standard concurrency + permissions stanza)

Step 5d Carry-Forwards Re-Verified

First CI for this repo. Mirrors the fleet pattern shipped this session
(better-budget #41, stock-trainer #35, Better-legacy #36, bookmarks #27,
save-for-later #38, flyck #61, better-bible-trivia #34, better-pastoral-
care #27, tennis-story #76): ubuntu-latest runner, runs `bash -n` syntax
check on every shell file, ShellCheck informationally (until issue #21's
.shellcheckrc policy decision lands; otherwise the 234-finding baseline
would block all PRs), and the existing tests/test_sandbox.sh smoke suite.

Logs `shellcheck --version` + `bash --version` so future baseline drifts
are visible (issue #21 was caused by an invisible shellcheck version
bump).

No CI existed prior to this commit — every change relied on local runs.
@pstuart pstuart self-assigned this May 5, 2026
@pstuart

pstuart commented May 5, 2026

Copy link
Copy Markdown
Owner Author

Reverting per author request — CI workflows weren't wanted; this PR is closed without merge.

@pstuart pstuart closed this May 5, 2026
@pstuart pstuart deleted the fix/auto-scan-20260505-ci branch May 5, 2026 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant