We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
A recent change added this line to uv-resolver/src/lock.rs:
uv-resolver/src/lock.rs
let lock_path = relative_to(workspace.lock_path(), &lock_path).unwrap();
This line panicks if workspace.lock_path is initialized from Path::new("").to_path_buf(), which does happen here
workspace.lock_path
Path::new("").to_path_buf()
Self::from_project( project_root, Path::new(""), &project, &pyproject_toml, options, )
and here.
let project = ProjectWorkspace::from_project( project_root, Path::new(""), project, &pyproject_toml, options, )
I'm not sure if this ever actually triggers in the existing codebase, but it seems inconsistent, probably worth a look?
The text was updated successfully, but these errors were encountered:
I can double-check, thanks.
Sorry, something went wrong.
FWIW replacing Path::new("") with project_root takes care of it and doesn't break any existing unit tests
Path::new("")
project_root
lock_path
d643e92
charliermarsh
Successfully merging a pull request may close this issue.
A recent change added this line to
uv-resolver/src/lock.rs
:This line panicks if
workspace.lock_path
is initialized fromPath::new("").to_path_buf()
, which does happen hereand here.
I'm not sure if this ever actually triggers in the existing codebase, but it seems inconsistent, probably worth a look?
The text was updated successfully, but these errors were encountered: