Skip to content

chore: Migrate from poetry to uv for the package and all examples#335

Merged
BulkBeing merged 7 commits intomainfrom
migrate-to-uv
Mar 16, 2026
Merged

chore: Migrate from poetry to uv for the package and all examples#335
BulkBeing merged 7 commits intomainfrom
migrate-to-uv

Conversation

@BulkBeing
Copy link
Contributor

@BulkBeing BulkBeing commented Mar 16, 2026

  • This PR doesn't change pynumaflow_lite. I will make the changes in a separate PR.
  • Made the example Dockerfiles simpler (removed dumbinit, using uv images itself as final image)
  • I think its better to store uv.lock file as well. Fastapi and pydantic does it. Reason: Unignore Cargo.lock file tokio-rs/axum#3104

Almost all part of the CI including pypi publish (only upto package build, and image builds. package/image publishing step was skipped) were tested https://github.com/BulkBeing/numaflow-python/pull/1/checks?sha=9a0656ad44fc0027e8f7a167440484baee754ab6

Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
Signed-off-by: Sreekanth <prsreekanth920@gmail.com>
@codecov
Copy link

codecov bot commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.31%. Comparing base (5b458ad) to head (445f7ba).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #335      +/-   ##
==========================================
- Coverage   94.40%   94.31%   -0.10%     
==========================================
  Files          67       67              
  Lines        3182     3182              
  Branches      179      179              
==========================================
- Hits         3004     3001       -3     
- Misses        143      146       +3     
  Partials       35       35              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@BulkBeing BulkBeing requested a review from yhl25 March 16, 2026 01:01
@vigith vigith changed the title Migrate from poetry to uv for the package and all examples chore: Migrate from poetry to uv for the package and all examples Mar 16, 2026
Copy link
Member

@vigith vigith left a comment

Choose a reason for hiding this comment

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

do we need these uv.lock?

@BulkBeing
Copy link
Contributor Author

do we need these uv.lock?

I think its good for reproducible builds. Fastapi and pyadantic is storing it in Git.

@BulkBeing BulkBeing requested a review from vigith March 16, 2026 02:48
@BulkBeing BulkBeing merged commit febbe7a into main Mar 16, 2026
11 checks passed
@BulkBeing BulkBeing deleted the migrate-to-uv branch March 16, 2026 03:36
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.

2 participants