Skip to content

swap around what we're building for docker#3490

Merged
winglian merged 2 commits into
mainfrom
no-uv-cloud-base
Mar 12, 2026
Merged

swap around what we're building for docker#3490
winglian merged 2 commits into
mainfrom
no-uv-cloud-base

Conversation

@winglian
Copy link
Copy Markdown
Collaborator

@winglian winglian commented Mar 12, 2026

Description

Motivation and Context

How has this been tested?

AI Usage Disclaimer

Screenshots (if appropriate)

Types of changes

Social Handles (Optional)

Summary by CodeRabbit

  • Chores
    • Added Python 3.12 support to build and test configurations.
    • Expanded platform matrices for additional Linux architectures (amd64, arm64).
    • Updated CI/CD pipeline strategy settings to adjust test execution behavior.

@winglian winglian merged commit 819b157 into main Mar 12, 2026
@winglian winglian deleted the no-uv-cloud-base branch March 12, 2026 01:45
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 12, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 59c35a18-340c-4168-8c4d-35511411993b

📥 Commits

Reviewing files that changed from the base of the PR and between fccc712 and 1cbd9b9.

📒 Files selected for processing (1)
  • .github/workflows/main.yml

📝 Walkthrough

Walkthrough

Modified .github/workflows/main.yml to add fail-fast: false to multiple workflow job strategy blocks, expanded platform matrices with explicit entries (linux/amd64, linux/arm64), introduced Python 3.12 configurations for specific CUDA/PyTorch combinations, and adjusted matrix field ordering.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/main.yml
Added fail-fast: false to strategy blocks; expanded platform matrix with explicit platform entries; introduced Python 3.12 matrix entries for specific CUDA/PyTorch combinations; removed older mixed entries; reordered matrix fields.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • SalmanMohammadi
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch no-uv-cloud-base

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

1 participant