From 6692d6c1e9efabe85fe16634b6aa4972479b7d01 Mon Sep 17 00:00:00 2001 From: Christopher Horrell Date: Mon, 25 May 2026 23:13:10 -0400 Subject: [PATCH] ci: replace Dockerized shfmt with direct binary install The mvdan/shfmt:v3 Docker image is not version-pinned and requires a docker pull on every PR. Install the shfmt binary directly from GitHub releases instead, matching the version used in .pre-commit-config.yaml (v3.13.1). Avoids a container pull while keeping behaviour identical. --- .github/workflows/linting.yml | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/.github/workflows/linting.yml b/.github/workflows/linting.yml index 9f706ce..a71e1c5 100644 --- a/.github/workflows/linting.yml +++ b/.github/workflows/linting.yml @@ -15,9 +15,17 @@ jobs: with: persist-credentials: false + - name: Install shfmt + env: + SHFMT_VERSION: "3.13.1" + run: | + curl -fsSL "https://github.com/mvdan/sh/releases/download/v${SHFMT_VERSION}/shfmt_v${SHFMT_VERSION}_linux_amd64" \ + -o /usr/local/bin/shfmt + chmod +x /usr/local/bin/shfmt + - name: shfmt run: | - docker run --rm -v "$(pwd)":/sh -w /sh mvdan/shfmt:v3 -sr -i 2 -l -w -ci . + shfmt -sr -i 2 -l -w -ci . git diff --color --exit-code shellcheck: