Skip to content

ci(framework): Add uv support#6679

Merged
danieljanes merged 41 commits intomainfrom
migrate-add-uv
Mar 6, 2026
Merged

ci(framework): Add uv support#6679
danieljanes merged 41 commits intomainfrom
migrate-add-uv

Conversation

@danieljanes
Copy link
Member

@danieljanes danieljanes commented Mar 3, 2026

Merge after #6651

@danieljanes danieljanes requested a review from tanertopal as a code owner March 3, 2026 19:06
@github-actions github-actions bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Mar 3, 2026
Base automatically changed from migrate-pyproject to main March 3, 2026 19:25
@danieljanes danieljanes marked this pull request as draft March 3, 2026 19:57
@danieljanes danieljanes marked this pull request as ready for review March 4, 2026 13:26
Copilot AI review requested due to automatic review settings March 4, 2026 13:26
Copy link
Contributor

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

Adds initial uv support for the framework/ Python project by documenting common uv workflows, wiring uv configuration into framework/pyproject.toml, and adjusting dev scripts to behave under uv run.

Changes:

  • Add framework/uv.md usage docs (sync/format/test/build) and a uv build helper script.
  • Configure uv defaults and local devtool source in framework/pyproject.toml, and set Coverage’s data file location.
  • Update framework/dev/test.sh to disable Ray’s uv run runtime-env hook during pytest.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
framework/uv.md Documents how to use uv with the framework project (sync/run/build).
framework/pyproject.toml Adds [tool.uv] config, local source mapping for devtool, and coverage data file location.
framework/dev/test.sh Sets an env var to avoid Ray stalling under uv run during pytest.
framework/dev/build-uv.sh Adds a small helper to build the framework using uv build.
dev/bootstrap-uv.sh Adds a uv bootstrap script under repo-root dev/ (currently references non-existent paths from that location).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

danieljanes and others added 3 commits March 4, 2026 15:34
@danieljanes danieljanes enabled auto-merge (squash) March 4, 2026 20:47
Copy link
Member

@chongshenng chongshenng left a comment

Choose a reason for hiding this comment

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

LGTM!

@danieljanes danieljanes merged commit bbea933 into main Mar 6, 2026
70 checks passed
@danieljanes danieljanes deleted the migrate-add-uv branch March 6, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainer Used to determine what PRs (mainly) come from Flower maintainers.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants