Skip to content

Commit da78ec1

Browse files
committed
Revert "Try to simplify none string logic"
This reverts commit c973ff4.
1 parent c973ff4 commit da78ec1

2 files changed

Lines changed: 11 additions & 5 deletions

File tree

.github/actions/install-deps/action.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ description: Install package dependencies using uv
33

44
inputs:
55
options:
6-
description: 'Package options to install (space-separated, e.g., "full").'
6+
description: 'Package options to install (space-separated, e.g., "full"). "none is an alias for empty string.'
77
required: false
88
default: 'full'
99
groups:
@@ -17,12 +17,18 @@ runs:
1717
- name: Install dependencies (options=[${{ inputs.options }}], groups=[${{ inputs.groups }}])
1818
shell: bash
1919
run: |
20+
# Handle "none" as alias for empty string
21+
options_input="${{ inputs.options }}"
22+
if [ "$options_input" = "none" ]; then
23+
options_input=""
24+
fi
25+
2026
# Build the install command
2127
cmd="uv pip install -e"
2228
2329
# Convert space-separated options to comma-separated for brackets
2430
# E.g. "abc def" => '.[abc,def]'
25-
options=$(echo "${{ inputs.options }}" | tr ' ' ',')
31+
options=$(echo "$options_input" | tr ' ' ',')
2632
cmd="$cmd '.[$options]'"
2733
2834
# Add groups

.github/workflows/tests.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
tests:
1414
# Default config: py3.14, ubuntu-latest, float32, full options.
1515
# The idea is to make each of those params vary one by one, to limit the number of tests to run.
16-
name: Run tests (py${{ matrix.python-version || '3.14' }}, ${{ matrix.os || 'ubuntu-latest' }}, ${{ matrix.dtype || 'float32' }}, ${{ matrix.options == null && 'full' || matrix.options }})
16+
name: Run tests (py${{ matrix.python-version || '3.14' }}, ${{ matrix.os || 'ubuntu-latest' }}, ${{ matrix.dtype || 'float32' }}, ${{ matrix.options || 'full' }})
1717
runs-on: ${{ matrix.os || 'ubuntu-latest' }}
1818
strategy:
1919
fail-fast: false
@@ -31,7 +31,7 @@ jobs:
3131
# dtype variations
3232
- dtype: float64
3333
# Installation options variations
34-
- options: ''
34+
- options: 'none'
3535

3636
steps:
3737
- name: Checkout repository
@@ -44,7 +44,7 @@ jobs:
4444

4545
- uses: ./.github/actions/install-deps
4646
with:
47-
options: ${{ matrix.options == null && 'full' || matrix.options }}
47+
options: ${{ matrix.options || 'full' }}
4848
groups: test
4949

5050
- name: Run tests

0 commit comments

Comments
 (0)