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

Use skip_install=true for tox.ini dev environment #235

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

hf-kklein
Copy link
Contributor

so that the package itself is not installed into the site packages which is rather annoying to debug

so that the package itself is not installed into the site packages which is rather annoying to debug
@hf-kklein hf-kklein requested review from a team, lord-haffi and hf-krechan October 9, 2024 12:29
@lord-haffi
Copy link
Contributor

It would be very inconvenient if tox is not installed into the dev environment.

@hf-kklein
Copy link
Contributor Author

It would be very inconvenient if tox is not installed into the dev environment.

is the PR title misleading? It's not skipping the installation of tox, it's about not installing the project under test in that env.

https://tox.wiki/en/latest/config.html#skip_install

Assume (in this repo) I have a src/my_project, then I hope to achieve, that with the changes from this PR, tox -e dev no longer installs my_project in the site packages although pip install .[dev] would do so.

at least that's my intention ;) I will test tomorrow if that's actually what it does.

Konstantin added 2 commits October 10, 2024 08:43
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