From 02138f57285f2ad1600ac75709a907c369bb035b Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Sat, 11 Apr 2026 16:31:36 -0400 Subject: [PATCH] =?UTF-8?q?fix(ci):=20use=20venv=20for=20ruff=20install=20?= =?UTF-8?q?=E2=80=94=20pipx=20not=20on=20runner=20image?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The py3.12-node22 runner doesn't ship pipx, so the previous commit's pipx install ruff failed with command not found. Switched to the same venv pattern the test job uses. --- .forgejo/workflows/ci.yml | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/.forgejo/workflows/ci.yml b/.forgejo/workflows/ci.yml index af9973a..084e3fa 100644 --- a/.forgejo/workflows/ci.yml +++ b/.forgejo/workflows/ci.yml @@ -76,12 +76,15 @@ jobs: - uses: actions/checkout@v6 - name: Install ruff - # pipx keeps ruff isolated instead of polluting the system - # site-packages (the old --break-system-packages was a smell). - run: pipx install ruff + # Isolated venv avoids polluting the runner's system + # site-packages (the old --break-system-packages was a smell) + # and pipx isn't available on the py3.12-node22 runner image. + run: | + python3.12 -m venv /opt/ruff-venv + /opt/ruff-venv/bin/pip install --quiet ruff - name: Lint - run: ruff check src/ + run: /opt/ruff-venv/bin/ruff check src/ test: name: Python tests