Skip to content

ci: unpin pixi#633

Merged
cpcloud merged 2 commits intoNVIDIA:mainfrom
cpcloud:unpin-pixi
Dec 3, 2025
Merged

ci: unpin pixi#633
cpcloud merged 2 commits intoNVIDIA:mainfrom
cpcloud:unpin-pixi

Conversation

@cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Dec 3, 2025

Update pixi lockfile to 0.60.0 version and unpin pixi in CI

@copy-pr-bot
Copy link

copy-pr-bot bot commented Dec 3, 2025

Auto-sync is disabled for ready for review pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@cpcloud cpcloud changed the title unpin pixi ci: unpin pixi Dec 3, 2025
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 3, 2025

Greptile Overview

Greptile Summary

Unpinned pixi version in CI workflows and updated pixi.lock to 0.60.0 format, resolving the lockfile compatibility issue.

  • Removed pixi-version: "v0.59.0" pin from all three workflow files (conda-python-tests, docs-build, simulator-test)
  • Updated pixi.lock to version 6 format with new pypi-prerelease-mode: if-necessary-or-explicit option for all environments
  • Bumped ml_dtypes dependency from 0.5.3 to 0.5.4 across all Python versions and architectures

Confidence Score: 5/5

  • Safe to merge - reverts temporary workaround and properly updates lockfile
  • This PR cleanly reverses a temporary pin that was added to work around a lockfile compatibility issue, and updates the lockfile to the new pixi 0.60.0 format. The changes are straightforward: removing version pins from CI workflows and updating the lockfile with the new format and minor dependency updates
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
.github/workflows/conda-python-tests.yaml 5/5 Removed pixi-version: "v0.59.0" pin, allowing setup-pixi action to use latest compatible version
.github/workflows/docs-build.yaml 5/5 Removed pixi-version: "v0.59.0" pin, allowing setup-pixi action to use latest compatible version
.github/workflows/simulator-test.yaml 5/5 Removed pixi-version: "v0.59.0" pin, allowing setup-pixi action to use latest compatible version
pixi.lock 5/5 Updated to pixi 0.60.0 format with new pypi-prerelease-mode option and ml_dtypes bumped from 0.5.3 to 0.5.4

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant GH as GitHub Actions
    participant Pixi as setup-pixi Action
    participant Lock as pixi.lock

    Note over Dev,Lock: Previous State (PR #632)
    Dev->>GH: Pinned pixi to v0.59.0
    Note over GH: pixi.lock incompatible with v0.60.0
    
    Note over Dev,Lock: Current PR #633
    Dev->>Lock: Update to pixi 0.60.0 format
    Note over Lock: Add pypi-prerelease-mode option<br/>Bump ml_dtypes 0.5.3 → 0.5.4
    Dev->>GH: Remove pixi-version pin
    
    Note over Dev,Lock: CI Workflow Execution
    GH->>Pixi: Setup pixi (no version specified)
    Pixi->>Pixi: Use latest compatible version (0.60.0+)
    Pixi->>Lock: Read pixi.lock
    Lock-->>Pixi: Version 6 format with new options
    Pixi->>GH: Environment configured successfully
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@cpcloud
Copy link
Contributor Author

cpcloud commented Dec 3, 2025

/ok to test

@cpcloud cpcloud enabled auto-merge (squash) December 3, 2025 17:02
@cpcloud cpcloud merged commit c1ead34 into NVIDIA:main Dec 3, 2025
140 of 142 checks passed
@cpcloud cpcloud deleted the unpin-pixi branch December 5, 2025 17:45
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