Skip to content

Commit ee8d4a6

Browse files
oschwaldclaude
andcommitted
Simplify CI tox steps now that lock runner is used
The uv-venv-lock-runner disregards --skip-pkg-install, making the two-step pattern (--notest then --skip-pkg-install) unnecessary. Consolidate into a single tox run step. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 68bb667 commit ee8d4a6

2 files changed

Lines changed: 2 additions & 12 deletions

File tree

.github/workflows/test-libmaxminddb.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,12 +60,7 @@ jobs:
6060
env:
6161
CFLAGS: "${{ env.CFLAGS }} -Werror -Wall -Wextra"
6262

63-
- name: Setup test suite
64-
run: tox run -vv --notest --skip-missing-interpreters false
65-
env:
66-
TOX_GH_MAJOR_MINOR: ${{ matrix.env }}
67-
6863
- name: Run test suite
69-
run: tox run --skip-pkg-install
64+
run: tox run --skip-missing-interpreters false
7065
env:
7166
TOX_GH_MAJOR_MINOR: ${{ matrix.env }}

.github/workflows/test.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,8 @@ jobs:
2929
- name: Install Python
3030
if: matrix.env != '3.14'
3131
run: uv python install --python-preference only-managed ${{ matrix.env }}
32-
- name: Setup test suite
33-
run: tox run -vv --notest --skip-missing-interpreters false
34-
env:
35-
MAXMINDDB_REQUIRE_EXTENSION: 1
36-
TOX_GH_MAJOR_MINOR: ${{ matrix.env }}
3732
- name: Run test suite
38-
run: tox run --skip-pkg-install
33+
run: tox run --skip-missing-interpreters false
3934
env:
4035
MAXMINDDB_REQUIRE_EXTENSION: 1
4136
MM_FORCE_EXT_TESTS: 1

0 commit comments

Comments
 (0)