Add highest and lowest dependency testing in CI #44
Conversation
There was a problem hiding this comment.
Pull request overview
This PR enhances the CI workflow to test the AlphaFlow package against both the highest and lowest compatible dependency versions, improving confidence that the package works across its supported dependency range.
Key Changes:
- Adds a matrix dimension for dependency resolution strategies (highest/lowest) to the test job
- Updates the test job name to include the resolution strategy being tested
- Modifies the dependency installation command to use the resolution strategy from the matrix
.github/workflows/ci.yml
Outdated
| fail-fast: false | ||
| matrix: | ||
| python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"] | ||
| resolution: [highest, lowest] |
There was a problem hiding this comment.
The value 'lowest' may not be a valid resolution strategy for uv. Based on uv's documentation, the valid values for the --resolution flag are 'highest' and 'lowest-direct', not 'lowest'. Consider changing this to 'lowest-direct' to ensure the CI correctly tests with the lowest compatible dependency versions.
| resolution: [highest, lowest] | |
| resolution: [highest, lowest-direct] |
brandonschabell
force-pushed
the
dependency-versions
branch
3 times, most recently
from
3a055ee to
9a57c5a
Compare
.github/workflows/ci.yml
Outdated
| run: uv sync --dev --resolution=${{ matrix.resolution }} | ||
| - name: Run ruff linting | ||
| run: uv run ruff check | ||
| run: uv run --no-sync ruff check |
There was a problem hiding this comment.
Extra space detected in the ruff check command. There are two spaces between "run" and "--no-sync" instead of one.
| run: uv run --no-sync ruff check | |
| run: uv run --no-sync ruff check |
brandonschabell
force-pushed
the
dependency-versions
branch
from
9a57c5a to
d8613e2
Compare
No description provided.