From 631d187f8a7e550d408a3af2db806aba185eddf7 Mon Sep 17 00:00:00 2001 From: Matt Sutton Date: Thu, 7 May 2026 09:01:52 -0400 Subject: [PATCH] Consolidate clippy + integration tests into one CI job Both jobs built the same tester image; merging them halves the docker build work. Also drop the pull_request gate so it runs on main pushes. Co-Authored-By: Claude Opus 4.7 (1M context) --- .github/workflows/ci.yml | 21 ++------------------- 1 file changed, 2 insertions(+), 19 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 32bda52..6eb34af 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -9,7 +9,6 @@ on: jobs: fmt: name: rustfmt - if: github.event_name == 'pull_request' runs-on: ubuntu-latest steps: - uses: actions/checkout@v6 @@ -18,25 +17,8 @@ jobs: components: rustfmt - run: cargo fmt --all -- --check - clippy: - name: clippy - if: github.event_name == 'pull_request' - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v6 - - uses: docker/setup-buildx-action@v4 - - uses: docker/build-push-action@v7 - with: - context: . - target: tester - load: true - tags: docoxide-tester:ci - cache-from: type=gha - cache-to: type=gha,mode=max - - run: docker run --rm docoxide-tester:ci cargo clippy --all-targets -- -D warnings - test: - name: integration tests + name: tests runs-on: ubuntu-latest steps: - uses: actions/checkout@v6 @@ -49,4 +31,5 @@ jobs: tags: docoxide-tester:ci cache-from: type=gha cache-to: type=gha,mode=max + - run: docker run --rm docoxide-tester:ci cargo clippy --all-targets -- -D warnings - run: docker run --rm --cap-add=SYS_NICE docoxide-tester:ci