Skip to content

ewoks install: support the poetry package manager - #330

Merged
woutdenolf merged 1 commit into
mainfrom
ewoks_install_freshenv
Sep 29, 2026
Merged

woutdenolf merged 1 commit into
mainfrom
ewoks_install_freshenv

Conversation

@woutdenolf

@woutdenolf woutdenolf commented Aug 6, 2026 •

Copy link
Copy Markdown
Member

ewoks convert and ewoks install support pip-venv, uv, pixi and conda. This PR adds poetry.

@woutdenolf woutdenolf mentioned this pull request Aug 6, 2026
@woutdenolf
woutdenolf force-pushed the ewoks_install_freshenv branch from 4567fb3 to 3a91ed9 Compare August 7, 2026 14:07
@codecov

codecov Bot commented Aug 7, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.56604% with 10 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/ewoks/_requirements/poetry.py 90.47% 10 Missing ⚠️

📢 Thoughts on this report? Let us know!

@woutdenolf
woutdenolf force-pushed the ewoks_install_freshenv branch 8 times, most recently from 9870073 to 758e36c Compare August 10, 2026 10:05
@woutdenolf
woutdenolf force-pushed the ewoks_install_freshenv branch 3 times, most recently from c5636db to 234f731 Compare August 19, 2026 15:22
@woutdenolf
woutdenolf force-pushed the ewoks_install_freshenv branch from 234f731 to 89b6ac0 Compare August 28, 2026 09:28
@woutdenolf
woutdenolf force-pushed the ewoks_install_freshenv branch from 89b6ac0 to a3c5588 Compare September 12, 2026 20:17
@woutdenolf
woutdenolf force-pushed the ewoks_install_freshenv branch from a3c5588 to 0a0978e Compare September 29, 2026 08:18
@woutdenolf woutdenolf changed the title Ewoks install freshenv ewoks install: support the poetry package manager Sep 29, 2026
Comment on lines +56 to +59
The project is generated from the installed python distributions instead of
being taken from a poetry project of the current environment. This keeps it
self-contained: no local project to build.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Just like the other lock file based managers we might change this (if possible). The docs describe the current implementation.

@woutdenolf
woutdenolf marked this pull request as ready for review September 29, 2026 08:25
@woutdenolf
woutdenolf requested a review from a team September 29, 2026 08:25
@woutdenolf
woutdenolf merged commit 6dc6fd9 into main Sep 29, 2026
12 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.

2 participants