Skip to content

Use tool.uv.sources to specify in situ plugin git dependency in pyproject #1209

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
May 20, 2025
Merged

Use tool.uv.sources to specify in situ plugin git dependency in pyproject #1209

merged 4 commits into from
May 20, 2025

Conversation

ml-evs
Copy link
Member

@ml-evs ml-evs commented May 20, 2025 •
edited
Loading

Closes #1208.

Also fixes the minimum setuptools version, and properly passes the README to the PyPI build process.

@ml-evs ml-evs added the build For issues/PRs pertaining to the build or deployment of the package label May 20, 2025
@codecov Codecov
Copy link

codecov bot commented May 20, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.09%. Comparing base (f3b47a0) to head (da78841).
⚠️ Report is 75 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #1209   +/-   ##
=======================================
  Coverage   71.09%   71.09%           
=======================================
  Files          65       65           
  Lines        4300     4300           
=======================================
  Hits         3057     3057           
  Misses       1243     1243
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cypress Cypress
Copy link

cypress bot commented May 20, 2025 •
edited
Loading

datalab    Run #3370

Run Properties:  status check passed Passed #3370  •  git commit 1ec9d6011b ℹ️: Merge dd6b7ef31616b83b42c5ab2d5bb203e7b3a6474d into f3b47a0356210b5acae43080359c...
Project datalab
Branch Review ml-evs/tweak-insitu-dep
Run status status check passed Passed #3370
Run duration 07m 29s
Commit git commit 1ec9d6011b ℹ️: Merge dd6b7ef31616b83b42c5ab2d5bb203e7b3a6474d into f3b47a0356210b5acae43080359c...
Committer Matthew Evans
View all properties for this run ↗︎
Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 504
View all changes introduced in this branch ↗︎

ml-evs
ml-evs commented May 20, 2025
View reviewed changes
@ml-evs ml-evs merged commit b53a868 into main May 20, 2025
17 checks passed
@ml-evs ml-evs deleted the ml-evs/tweak-insitu-dep branch May 20, 2025 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdbocarsly jdbocarsly Awaiting requested review from jdbocarsly jdbocarsly is a code owner

@BenjaminCharmes BenjaminCharmes Awaiting requested review from BenjaminCharmes BenjaminCharmes is a code owner

Assignees
No one assigned
Labels
build For issues/PRs pertaining to the build or deployment of the package
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Cannot release to PyPI with direct git dependency on in situ plugin
1 participant
@ml-evs