ewoks install: support the pixi package manager - #337
Conversation
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
| def test_create_environment(manager_case, environment): | ||
| assert environment.exists() | ||
| assert environment.python_version() == PYTHON_VERSION | ||
| if manager_case.CHANNEL_PYTHON: | ||
| assert environment.python_version().startswith(f"{PYTHON_MINOR_VERSION}.") | ||
| else: | ||
| assert environment.python_version() == PYTHON_VERSION |
There was a problem hiding this comment.
This one was tricky. The environment fixture creates an empty environment with the manager case (pip-venv, uv and pixi).
It uses platform.python_version() by default. But when using pixi and conda, the python comes from the conda channel and the version is not exactly the same, up to the minor. See python_specifier which is used to trim the patch version. This is needed because not all patch versions of python are in the conda channel.
| enable-coverage: ${{ matrix.enable-coverage }} | ||
| jupyter-platform-dirs: "1" | ||
| package-managers: "uv" | ||
| package-managers: "uv pixi" |
There was a problem hiding this comment.
When everything is done we can add something to CONTRIBUTING.md for what is needed to run the full test suite.
1 participant
PR summary
ewoks convertandewoks installsupport pip-venv and uv. This PR adds pixi.AI Disclosure