From da43559b7c333f57a826f005fb5719401667c7f5 Mon Sep 17 00:00:00 2001 From: Leif Denby Date: Tue, 9 Jun 2026 15:33:00 +0200 Subject: [PATCH] Fix pre-commit run in ci by including deps Fix workflow file to ensure graphviz and pre-commit is actually installed in CI when trying to run pre-commit --- .github/workflows/pre-commit.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/workflows/pre-commit.yml b/.github/workflows/pre-commit.yml index 85fed30..8d64cdf 100644 --- a/.github/workflows/pre-commit.yml +++ b/.github/workflows/pre-commit.yml @@ -18,6 +18,11 @@ jobs: with: enable-cache: true + # The local `config-diagram` hook renders the Fiddle config graph with + # graphviz, which shells out to the `dot` binary. + - run: sudo apt-get update && sudo apt-get install -y graphviz + # Run pre-commit through uv so CI uses the same project-managed environment # as local development, including local hooks that invoke `uv run ...`. - - run: uv run pre-commit run --all-files + # pre-commit lives in the `dev` extra, so it must be synced for `uv run`. + - run: uv run --extra dev pre-commit run --all-files