Skip to content

ci(datasets): Refactor uv setup#6690

Merged
danieljanes merged 2 commits intomainfrom
migrate-uv-setup
Mar 4, 2026
Merged

ci(datasets): Refactor uv setup#6690
danieljanes merged 2 commits intomainfrom
migrate-uv-setup

Conversation

@danieljanes
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 4, 2026 15:46
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

This PR refactors the datasets CI workflows to rely on the repository’s shared ./.github/actions/bootstrap composite action for installing/configuring uv, instead of installing uv directly in each workflow.

Changes:

  • Remove per-workflow astral-sh/setup-uv steps from datasets workflows.
  • Enable uv installation via the bootstrap action by setting uv-skip: "false".

Reviewed changes

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

File Description
.github/workflows/datasets-test.yml Uses the shared bootstrap action to install uv (via uv-skip: "false") instead of a dedicated setup-uv step.
.github/workflows/datasets-e2e.yml Same refactor as above for the E2E datasets workflow.

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

@danieljanes danieljanes enabled auto-merge (squash) March 4, 2026 15:50
Copy link
Member

@tanertopal tanertopal left a comment

Choose a reason for hiding this comment

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

Lgtm!

@github-actions github-actions bot added the Maintainer Used to determine what PRs (mainly) come from Flower maintainers. label Mar 4, 2026
@danieljanes danieljanes merged commit bd21db6 into main Mar 4, 2026
73 checks passed
@danieljanes danieljanes deleted the migrate-uv-setup branch March 4, 2026 16:21
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.

3 participants