From eaa005223b0342ff3e4a9ccdf8056435ab99d1c2 Mon Sep 17 00:00:00 2001 From: Matthew Alexander LaChance Date: Tue, 17 Dec 2024 10:27:58 -0500 Subject: [PATCH 1/2] GitHub Actions - run linting and specs separately --- .github/workflows/ci.yml | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index a7f5a3176..edb105b42 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -8,10 +8,15 @@ on: jobs: test: runs-on: ${{ contains( github.event.pull_request.labels.*.name, 'use_fast_ci') && 'ubuntu-latest-8core' || 'ubuntu-latest' }} + strategy: + fail-fast: false + matrix: + target: + - "lint" + - "specs" steps: - uses: actions/checkout@v4 - name: Run tests run: | docker compose --file docker-compose.test.yml build - docker compose --file docker-compose.test.yml run lint - docker compose --file docker-compose.test.yml run specs + docker compose --file docker-compose.test.yml run ${{ matrix.target }} From 4e1795fefa6139cca8bd4f07b82bd6f5574182d9 Mon Sep 17 00:00:00 2001 From: Ryan Rotter Date: Tue, 17 Dec 2024 13:43:07 -0500 Subject: [PATCH 2/2] Remove support for faster/paid CI runners Wasn't being used, and we really don't want to use paid runners for linting. --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index edb105b42..dd5ce3b5d 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -7,7 +7,7 @@ on: jobs: test: - runs-on: ${{ contains( github.event.pull_request.labels.*.name, 'use_fast_ci') && 'ubuntu-latest-8core' || 'ubuntu-latest' }} + runs-on: ubuntu-latest strategy: fail-fast: false matrix: