Skip to content

Commit dc35287

Browse files
committed
Update variants workflow for auto-detection
- Remove MXENV_UV_GLOBAL references (now auto-detected) - Update test names to reflect auto-detection behavior - Add new test for UV_PYTHON different from PRIMARY_PYTHON - All 8 scenarios now test auto-detection properly
1 parent a5a5e4a commit dc35287

File tree

1 file changed

+14
-8
lines changed

1 file changed

+14
-8
lines changed

.github/workflows/variants.yml

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,14 @@ jobs:
3030
run: |
3131
make VENV_ENABLED=true VENV_CREATE=true PRIMARY_PYTHON=python3 PYTHON_PACKAGE_INSTALLER=pip test
3232
make clean
33-
- name: VENV to be created with uv to be installed
33+
- name: VENV to be created with uv (local install, no global UV)
3434
run: |
35-
make VENV_ENABLED=true VENV_CREATE=true PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=python3 MXENV_UV_GLOBAL=false test
35+
make VENV_ENABLED=true VENV_CREATE=true PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=python3 test
3636
make clean
37-
- name: VENV to be created with uv globally pre-installed
37+
- name: VENV to be created with uv (global UV auto-detected)
3838
run: |
3939
pip install uv
40-
make VENV_ENABLED=true VENV_CREATE=true PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=${{ matrix.python-version }} MXENV_UV_GLOBAL=true test
40+
make VENV_ENABLED=true VENV_CREATE=true PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=${{ matrix.python-version }} test
4141
make clean
4242
pip uninstall -y uv
4343
- name: VENV pre-installed with pip
@@ -46,20 +46,26 @@ jobs:
4646
make VENV_ENABLED=true VENV_CREATE=false VENV_FOLDER=existingvenv PRIMARY_PYTHON=python3 PYTHON_PACKAGE_INSTALLER=pip test
4747
make clean
4848
rm -r existingvenv
49-
- name: VENV pre-installed with uv to be installed
49+
- name: VENV pre-installed with uv (local install, no global UV)
5050
run: |
5151
python -m venv existingvenv
52-
make VENV_ENABLED=true VENV_CREATE=false VENV_FOLDER=existingvenv PRIMARY_PYTHON=python3 PYTHON_PACKAGE_INSTALLER=uv MXENV_UV_GLOBAL=false test
52+
make VENV_ENABLED=true VENV_CREATE=false VENV_FOLDER=existingvenv PRIMARY_PYTHON=python3 PYTHON_PACKAGE_INSTALLER=uv test
5353
make clean
5454
rm -r existingvenv
55-
- name: VENV pre-installed with uv globally pre-installed
55+
- name: VENV pre-installed with uv (global UV auto-detected)
5656
run: |
5757
python -m venv existingvenv
5858
pip install uv
59-
make VENV_ENABLED=true VENV_CREATE=false VENV_FOLDER=existingvenv PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=python3 MXENV_UV_GLOBAL=true test
59+
make VENV_ENABLED=true VENV_CREATE=false VENV_FOLDER=existingvenv PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=python3 test
6060
make clean
6161
pip uninstall -y uv
6262
rm -r existingvenv
63+
- name: VENV with global UV using different UV_PYTHON
64+
run: |
65+
pip install uv
66+
make VENV_ENABLED=true VENV_CREATE=true PYTHON_PACKAGE_INSTALLER=uv PRIMARY_PYTHON=python3 UV_PYTHON=${{ matrix.python-version }} test
67+
make clean
68+
pip uninstall -y uv
6369
- name: Global Python with pip
6470
run: |
6571
make VENV_ENABLED=false VENV_CREATE=false PRIMARY_PYTHON=python3 PYTHON_PACKAGE_INSTALLER=pip test

0 commit comments

Comments
 (0)