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

Display warning if PIP_CONSTRAINT(S) are defined and UV_CONSTRAINTS not #154

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

ssbarnea
Copy link
Member

@ssbarnea ssbarnea force-pushed the fix/use-pip-constraints branch 2 times, most recently from f390073 to 134d164 Compare January 16, 2025 18:55
@ssbarnea ssbarnea changed the title Translate PIP_CONSTRAINT(S) into UV_CONSTRAINTS if needed Display warning if PIP_CONSTRAINT(S) are defined and UV_CONSTRAINTS not Jan 16, 2025
@ssbarnea ssbarnea marked this pull request as ready for review January 16, 2025 18:57
src/tox_uv/_venv.py Outdated Show resolved Hide resolved
tests/test_tox_uv_venv.py Outdated Show resolved Hide resolved
ssbarnea added a commit to ssbarnea/tox-uv that referenced this pull request Jan 16, 2025
gaborbernat pushed a commit that referenced this pull request Jan 16, 2025
@ssbarnea ssbarnea force-pushed the fix/use-pip-constraints branch from 134d164 to d159181 Compare January 16, 2025 22:31
@ssbarnea ssbarnea requested a review from gaborbernat January 16, 2025 22:34
tests/test_tox_uv_venv.py Outdated Show resolved Hide resolved
@ssbarnea ssbarnea force-pushed the fix/use-pip-constraints branch from d159181 to 9a75449 Compare January 16, 2025 22:54
@ssbarnea ssbarnea force-pushed the fix/use-pip-constraints branch 2 times, most recently from ff50287 to a8aaa65 Compare January 16, 2025 23:15
@ssbarnea ssbarnea force-pushed the fix/use-pip-constraints branch from a8aaa65 to 6e70ae2 Compare January 16, 2025 23:28
@ssbarnea ssbarnea force-pushed the fix/use-pip-constraints branch from 6e70ae2 to fcd6135 Compare January 16, 2025 23:47
@gaborbernat gaborbernat merged commit 87daeb6 into tox-dev:main Jan 16, 2025
9 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