Skip to content

Support uv - #328

Draft
woutdenolf wants to merge 2 commits into
mainfrom
support_uv
Draft

Support uv#328
woutdenolf wants to merge 2 commits into
mainfrom
support_uv

Conversation

@woutdenolf

Copy link
Copy Markdown
Member

Continue with #41. This PR adds UV and refactors tests. AI warning.

@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

@woutdenolf

Copy link
Copy Markdown
Member Author

@loichuder Not done yet. But I'm not familiar with uv so I'm not sure using uv in this way makes sense. Perhaps I should focus on creating fresh environments and drop support for installation in the current environment?

@woutdenolf

woutdenolf commented Aug 6, 2026

Copy link
Copy Markdown
Member Author

To be more specific, we use uv pip here (uv pip install, uv pip freeze) which is why we have a common base logic with pip in the refactored pip/uv managers and tests.

If we refactor and drop installation in the current environment then we have "pip+venv" (native), "uv", "pixi", "conda", "poetry" (we can drop "pipenv").

@loichuder

Copy link
Copy Markdown
Member

Discussed in-person. Based on the fact that most modern package managers abstract away virtual env management, we decided that ewoks install should do the same. That means we should not have to enter manually in a venv: ewoks install should create a new env for us or use whatever machinery the underlying packing manager is using to abstract the venv.

However, to cater for dev/debugging usecases, we will add a flag --in-place to keep the current behaviour (that is, installing in the current env) for pip and uv.

@woutdenolf

Copy link
Copy Markdown
Member Author

Abandoned this PR in favor of what was discussed.

Current WIP #330 to be split in parts when finished.

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