Skip to content

Conversation

@maxisbey
Copy link
Contributor

Summary

Replace the highest dependency resolution strategy with locked in the CI test matrix.

Motivation and Context

The highest resolution strategy was causing CI failures across multiple PRs because it was picking up ruff==0.14.12 which doesn't have wheels for:

  • win_amd64 (Windows x64)
  • Some Linux glibc versions (e.g., manylinux_2_39_x86_64)

This was causing failures like:

error: Distribution `ruff==0.14.12 @ registry+https://pypi.org/simple` can't be installed 
because it doesn't have a source distribution or wheel for the current platform

Changes

Replace highest with locked:

  • Before: Tests against lowest-direct and highest resolution
  • After: Tests against lowest-direct and locked (frozen lockfile versions)

This provides:

  1. Stability: Locked versions are known to work and have all platform wheels
  2. Minimum version testing: lowest-direct still verifies minimum supported versions
  3. Reproducibility: Tests run against the same versions developers use locally

Breaking Changes

None. This only affects CI configuration.

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines

The 'highest' resolution strategy was causing CI failures because it
was picking up ruff==0.14.12 which doesn't have wheels for win_amd64
and some Linux glibc versions.

Replace it with 'locked' which uses the frozen lockfile versions,
providing more stable CI while still testing lowest-direct for minimum
version compatibility.

Claude-Generated-By: Claude Code (cli/claude-opus-4-5=100%)
Claude-Steers: 1
Claude-Permission-Prompts: 1
Claude-Escapes: 0
@maxisbey maxisbey enabled auto-merge (squash) January 15, 2026 16:33
@Kludex Kludex disabled auto-merge January 16, 2026 10:09
@Kludex Kludex merged commit 2cf7784 into main Jan 16, 2026
26 checks passed
@Kludex Kludex deleted the ci-remove-highest-resolution branch January 16, 2026 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants