-
Notifications
You must be signed in to change notification settings - Fork 453
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
Add test case for wheel installation with different path #4396
Conversation
When you say "path" here, do you mean just the wheel's filename or the whole absolute path to it? I don't think the path where the wheel file is located should matter -
|
I think we're just using the full file path. I can add test coverage for just the name too. |
FWIW one case where I think using the full path would be weird is when you have a cronjob or similar that downloads a "latest" wheel from some place (GH actions build or similar) to a temporary location, which might be a In such a case I think the expected behavior would be to reinstall only if the version changed (or if I provide a switch to force reinstalling)... |
Here's the mentioned test case #4398 We don't do what you want yet :) |
…th (#4398) As discussed at #4396 (comment) This may not be the desired behavior, but let's track it.
…th (astral-sh#4398) As discussed at astral-sh#4396 (comment) This may not be the desired behavior, but let's track it.
Regression test for #4391 / #4393