-
Notifications
You must be signed in to change notification settings - Fork 175
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
feat: add --force-reinstall for global install #2275
feat: add --force-reinstall for global install #2275
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
@@ -545,6 +545,45 @@ def test_install_twice(pixi: Path, tmp_path: Path, dummy_channel_1: str) -> None | |||
assert dummy_b.is_file() | |||
|
|||
|
|||
def test_install_twice_with_force_reinstall( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You could change the default channel in the config after the first install and check that another install does nothing. Then run force reinstall and check that it worked
as discussed with @Hofer-Julian I will add in the follow-up PR the newer test that will also include this fix: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missed one thing before
Co-authored-by: Hofer-Julian <[email protected]>
Co-authored-by: Hofer-Julian <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 🎉
66b80c0
into
prefix-dev:feature/pixi-global
I thought that it would be nice to name it the same as we have it in
pixi exec