Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the uv ecosystem #11687

Merged
merged 4 commits into from
Feb 28, 2025
Merged

Add the uv ecosystem #11687

merged 4 commits into from
Feb 28, 2025

Conversation

markhallen
Copy link
Contributor

@markhallen markhallen commented Feb 27, 2025

What are you trying to accomplish?

Part of #10478

This PR build on the community submission #11687.

Anything you want to highlight for special attention from reviewers?

We Have split this PR into three commits:

  • Prepare dependabot for the uv ecosystem
    • Has all the code outside the ecosystem folders (./uv, ./python) to prepare for the new ecosystem
  • uv copy of the python folder and tested dry run
    • Is a copy of the python folder with uv handling added
  • Update uv to remove other python package managers
    • Cleans up the copied folder to only handle the uv package manager

How will you know you've accomplished your goal?

We can dry run with our uv testing repo:

./bin/docker-dev-shell uv --rebuild
./bin/dry-run.rb uv dsp-testing/dependabot-uv

Checklist

  • I have run the complete test suite to ensure all tests and linters pass.
  • I have thoroughly tested my code changes to ensure they work as expected, including adding additional tests for new functionality.
  • I have written clear and descriptive commit messages.
  • I have provided a detailed description of the changes in the pull request, including the problem it addresses, how it fixes the problem, and any relevant details about the implementation.
  • I have ensured that the code is well-documented and easy to understand.

@markhallen markhallen requested a review from a team as a code owner February 27, 2025 14:59
@markhallen markhallen force-pushed the markhallen/add-uv-ecosystem branch from 1dcd9c9 to 4c332fe Compare February 28, 2025 11:08
markhallen and others added 2 commits February 28, 2025 11:53
This commit is after copying the python folder and running:

./bin/docker-dev-shell uv --rebuild
./bin/dry-run.rb uv dsp-testing/dependabot-uv

Co-authored-by: Gastón Avila <avila.gas@gmail.com>
Co-authored-by: Edgar Ramírez Mondragón <16805946+edgarrmondragon@users.noreply.github.com>
Co-authored-by: skshetry <18718008+skshetry@users.noreply.github.com>
Co-authored-by: Albert Ferràs <albert.ferras@veriff.net>
Co-authored-by: beagold <86345081+beagold@users.noreply.github.com>
@markhallen markhallen force-pushed the markhallen/add-uv-ecosystem branch from 4c332fe to 1972be3 Compare February 28, 2025 11:54
Copy link
Contributor

@thavaahariharangit thavaahariharangit left a comment

Choose a reason for hiding this comment

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

I’m approving this PR based on the following reasons:

  1. This is an initial update, and any specific issues can be addressed in future PRs.
  2. It allows the developer to test the changes in other environments.
  3. Confident that the developer can successfully create a Docker image for the new UV ecosystem.
  4. The update includes 460+ files, all contained within their own module.
  5. This will help unblock the author and allow progress on the next tasks.

@markhallen markhallen enabled auto-merge February 28, 2025 14:25
@markhallen markhallen merged commit 021be4f into main Feb 28, 2025
176 of 184 checks passed
@markhallen markhallen deleted the markhallen/add-uv-ecosystem branch February 28, 2025 14:29
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