Skip to content

Switch to uv2nix#2345

Merged
niklasmohrin merged 5 commits intoe-valuation:mainfrom
niklasmohrin:uv
Dec 2, 2024
Merged

Switch to uv2nix#2345
niklasmohrin merged 5 commits intoe-valuation:mainfrom
niklasmohrin:uv

Conversation

@niklasmohrin
Copy link
Member

@niklasmohrin niklasmohrin commented Nov 25, 2024

The maintainer of poetry2nix has abandoned the project with recommendation to move to uv2nix instead.

When working on #2328 I also noticed that building distributions with poetry's build backend with the recommended python -m build method does not work as straightforward as I expected. With uv, hatch is the recommended build backend. We thus also move to this and now use more standardized fields of pyproject.toml.

@niklasmohrin
Copy link
Member Author

@ybrnr Can you test this on your computer? This time, it is nix develop github:niklasmohrin/evap/uv and nix run github:niklasmohrin/evap/uv#services-full; please check ./manage.py run and ./manage.py precommit :)

@ybrnr
Copy link
Collaborator

ybrnr commented Nov 25, 2024

@niklasmohrin I cannot run the services with nix, because there is no wheel for psycopg...
maybe you forgot to update this branch?

@niklasmohrin
Copy link
Member Author

Ah, I was hoping that this would maybe fix itself with this :/

remove comment

simplify

rm poetry.lock

remove hammer

make psycopg-binary optional again

good practice

Use psycopg-c on macos
@niklasmohrin
Copy link
Member Author

@ybrnr How about now?

@niklasmohrin niklasmohrin marked this pull request as ready for review December 2, 2024 18:18
@niklasmohrin niklasmohrin requested a review from Kakadus December 2, 2024 18:33
@niklasmohrin niklasmohrin changed the title Migrate to uv2nix Switch to uv2nix Dec 2, 2024
@niklasmohrin niklasmohrin merged commit 01d979f into e-valuation:main Dec 2, 2024
@niklasmohrin niklasmohrin deleted the uv branch December 2, 2024 20:23
jooooosef pushed a commit to jooooosef/EvaP that referenced this pull request Jan 13, 2025
I also added a CI job to build the nix setup on Mac, so we can catch missing wheels early on.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants