From 362b00cc12705ec6ffc4e07d171fecee71303711 Mon Sep 17 00:00:00 2001 From: Charlie Marsh Date: Mon, 3 Jun 2024 14:42:38 -0400 Subject: [PATCH] Remove need to return Python version in `get_dependencies` (#3993) ## Summary Once we use a _range_ rather than a precise version, it won't actually make sense to return a version here. It's no longer required, so I'm removing it. --- crates/uv-resolver/src/resolver/mod.rs | 21 ++++----------------- 1 file changed, 4 insertions(+), 17 deletions(-) diff --git a/crates/uv-resolver/src/resolver/mod.rs b/crates/uv-resolver/src/resolver/mod.rs index ecc4c7467..f4abb479d 100644 --- a/crates/uv-resolver/src/resolver/mod.rs +++ b/crates/uv-resolver/src/resolver/mod.rs @@ -2,7 +2,6 @@ use std::borrow::Cow; use std::fmt::{Display, Formatter}; -use std::ops::Deref; use std::sync::Arc; use std::thread; @@ -652,22 +651,10 @@ impl ResolverState { - let version = self.python_requirement.installed(); - if range.contains(version) { - Ok(Some(ResolverVersion::Available(version.deref().clone()))) - } else { - Ok(None) - } - } - - PubGrubPackageInner::Python(PubGrubPython::Target) => { - let version = self.python_requirement.target(); - if range.contains(version) { - Ok(Some(ResolverVersion::Available(version.deref().clone()))) - } else { - Ok(None) - } + PubGrubPackageInner::Python(_) => { + // Dependencies on Python are only added when a package is incompatible; as such, + // we don't need to do anything here. + Ok(None) } PubGrubPackageInner::Extra {