Skip to content

ci(shellcheck): install and run native shellcheck on ubuntu-latest#6

Merged
benrozsa merged 2 commits intomainfrom
chore/ci/shellcheck-native
Sep 28, 2025
Merged

ci(shellcheck): install and run native shellcheck on ubuntu-latest#6
benrozsa merged 2 commits intomainfrom
chore/ci/shellcheck-native

Conversation

@benrozsa
Copy link
Owner

@benrozsa benrozsa commented Sep 14, 2025

Summary

Replace third‑party ShellCheck action with native installation and direct invocation.

Why

  • Eliminates tag resolution errors (e.g., unable to resolve ludeeus/action-shellcheck@v2)
  • Removes dependency on external action tags; stable on ubuntu‑latest

Changes

  • Install ShellCheck via apt‑get on ubuntu‑latest
  • Run shellcheck -S error -x on tracked shell files
  • Trigger on .github/** so infra‑only PRs still satisfy the required check

Testing

  • CI runs successfully; when no shell files are present, ShellCheck exits cleanly and passes

Risks/Mitigations

  • None; behavior limited to CI pipeline and uses distro package

Replace ludeeus/action-shellcheck action with apt-based installation and direct invocation. Avoids tag resolution errors (e.g., 'unable to find version v2') and removes third-party action dependency.
@benrozsa benrozsa enabled auto-merge (squash) September 14, 2025 19:46
@benrozsa benrozsa merged commit 4a91eb6 into main Sep 28, 2025
2 checks passed
@benrozsa benrozsa deleted the chore/ci/shellcheck-native branch September 28, 2025 15:01
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