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

Ensure setuptools, wheel, and pip is present when initializing environment when using uv #59

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

cccs-rs
Copy link
Contributor

@cccs-rs cccs-rs commented Nov 13, 2024

Related to: #57

@seb-acsc
Copy link
Contributor

Unfortunately, doesn't seem to help with a fresh tox and virtual environment.
I'm on Ubuntu 22.04.4 LTS and seeing tests succeed on 3.8 & 3.9 but fail on 3.11 and 3.12 in tox.

@cccs-ml
Copy link
Contributor

cccs-ml commented Nov 14, 2024

I can also replicate outside of tox.

Creating a new venv where setuptools and wheel are not pre-installed. Install pytest, requirements.txt and tests/requirements.txt. Run python -m pytest.

@cccs-ml
Copy link
Contributor

cccs-ml commented Nov 15, 2024

Unclear why tests are passing using tox in the pipeline here.

I can still replicate the issue with:

docker run --rm -ti -v .:/src --entrypoint bash python:3.12-slim

apt update
apt install git
cd /src
python -m venv venv
source venv/bin/activate
pip install pytest
pip install -r requirements.txt
pip install -r tests/requirements.txt
python -m pytest

pip install tox
python -m tox -e py

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.

3 participants