Skip to content

Conversation

leohoare
Copy link
Contributor

@leohoare leohoare commented Jun 22, 2025

This PR

This PR migrates hatch to UV (keeping the build system as hatchling)

Related Issues

#502

Notes / Questions

  • Previously scripts were nicely run through hatch. How would we like to keep this functionality now we're migrating to UV?
  • Is this project okay with using astral-sh/setup-uv in github actions?

TODOs

  • Handle pre-commit
  • Sast
  • Finalise developer patterns / guides

Copy link

codecov bot commented Jun 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.65%. Comparing base (7783a8b) to head (200af7d).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #512   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files          37       37           
  Lines        1788     1788           
=======================================
  Hits         1746     1746           
  Misses         42       42           
Flag Coverage Δ
unittests 97.65% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

@leohoare leohoare changed the title starting migration to uv feat: starting migration to uv Jun 22, 2025
@leohoare leohoare mentioned this pull request Jun 22, 2025
Copy link
Member

@aepfli aepfli left a comment

Choose a reason for hiding this comment

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

Thank you! I am really looking forward to this. Especially if astral-sh/uv#6365 gets resolved we can add additional targets. Generally i am pretty happy with this starting point. We should focus on ease of use for contributors and IMHO make a lot of opinionated configurations upfront, to reduce the needed knowledge for adopters

leohoare and others added 6 commits June 23, 2025 22:14
Signed-off-by: leohoare <[email protected]>
… mypy and pre-commit to dev dependencies

Signed-off-by: leohoare <[email protected]>
Signed-off-by: leohoare <[email protected]>
Co-authored-by: Simon Schrottner <[email protected]>
Signed-off-by: Leo  <[email protected]>
Signed-off-by: leohoare <[email protected]>
@leohoare leohoare force-pushed the feature/migrate_to_uv branch from fc04db3 to 368bb40 Compare June 23, 2025 14:14
@leohoare leohoare marked this pull request as ready for review June 25, 2025 00:44
@leohoare leohoare requested review from a team as code owners June 25, 2025 00:44
@beeme1mr beeme1mr requested review from federicobond and gruebel June 25, 2025 14:47
Copy link
Member

@gruebel gruebel left a comment

Choose a reason for hiding this comment

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

thanks, great work overall. left a few comments, but nothing too critical.

Copy link
Member

@gruebel gruebel left a comment

Choose a reason for hiding this comment

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

thanks 🍻

@gruebel gruebel merged commit fb47cbb into open-feature:main Jun 27, 2025
15 checks passed
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