Skip to content

Conversation

@barjin
Copy link
Member

@barjin barjin commented Oct 17, 2025

closes #286

@barjin barjin requested a review from Copilot October 17, 2025 09:13
@barjin barjin self-assigned this Oct 17, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds the --frozen flag to all uv sync commands in CI workflows to ensure consistent dependency installations. The change enforces that package installations must match the lockfile exactly, preventing unexpected dependency updates during CI runs.

  • Adds --frozen flag to uv sync commands across all Python CI workflows
  • Ensures deterministic builds by preventing lockfile modifications during CI
  • Applies the change consistently to test, code check, and documentation workflows

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
.github/workflows/python-test.yaml Added --frozen flag to 5 uv sync commands across different test job configurations
.github/workflows/python-code-checks.yaml Added --frozen flag to 2 uv sync commands in lint and type check jobs
.github/workflows/docs.yaml Added --frozen flag to 1 uv sync command in documentation build job

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added this to the 125th sprint - Tooling team milestone Oct 17, 2025
@github-actions github-actions bot added the t-tooling Issues with this label are in the ownership of the tooling team. label Oct 17, 2025
@barjin
Copy link
Member Author

barjin commented Oct 17, 2025

Python build test running here.

@barjin barjin merged commit f3f6ad9 into master Oct 17, 2025
27 checks passed
@barjin barjin deleted the chore/fix-python-musllinux-builds branch October 17, 2025 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failing CI builds of Impit for Python on musllinux

1 participant