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

Switch to uv2nix #2345

Merged
merged 5 commits into from
Dec 2, 2024
Merged

Switch to uv2nix #2345

merged 5 commits into from
Dec 2, 2024

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 :)

flake.nix Outdated Show resolved Hide resolved
uv.lock Show resolved Hide resolved
@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
13 checks passed
@niklasmohrin niklasmohrin deleted the uv branch December 2, 2024 20:23
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