From c485727e00b06d5657c2269e7a46b40c849ebaeb Mon Sep 17 00:00:00 2001 From: Charlie Marsh Date: Wed, 21 Aug 2024 11:29:33 -0400 Subject: [PATCH] Remove duplicated lockfile invalidation logs (#6340) ## Summary These got moved to the caller, so they're all duplicated right now. --- crates/uv-resolver/src/lock.rs | 27 --------------------------- 1 file changed, 27 deletions(-) diff --git a/crates/uv-resolver/src/lock.rs b/crates/uv-resolver/src/lock.rs index 0d435d933..952d5b387 100644 --- a/crates/uv-resolver/src/lock.rs +++ b/crates/uv-resolver/src/lock.rs @@ -10,7 +10,6 @@ use itertools::Itertools; use petgraph::visit::EdgeRef; use rustc_hash::{FxHashMap, FxHashSet}; use toml_edit::{value, Array, ArrayOfTables, InlineTable, Item, Table, Value}; -use tracing::debug; use url::Url; use cache_key::RepositoryUrl; @@ -695,10 +694,6 @@ impl Lock { let expected = members.iter().cloned().collect::>(); let actual = &self.manifest.members; if expected != *actual { - debug!( - "Mismatched members:\n expected: {:?}\n found: {:?}", - expected, actual - ); return Ok(SatisfiesResult::MismatchedMembers(expected, actual)); } } @@ -718,10 +713,6 @@ impl Lock { .map(|requirement| normalize_requirement(requirement, workspace)) .collect::>()?; if expected != actual { - debug!( - "Mismatched requirements:\n expected: {:?}\n found: {:?}", - expected, actual - ); return Ok(SatisfiesResult::MismatchedConstraints(expected, actual)); } } @@ -741,10 +732,6 @@ impl Lock { .map(|requirement| normalize_requirement(requirement, workspace)) .collect::>()?; if expected != actual { - debug!( - "Mismatched constraints:\n expected: {:?}\n found: {:?}", - expected, actual - ); return Ok(SatisfiesResult::MismatchedConstraints(expected, actual)); } } @@ -764,10 +751,6 @@ impl Lock { .map(|requirement| normalize_requirement(requirement, workspace)) .collect::>()?; if expected != actual { - debug!( - "Mismatched overrides:\n expected: {:?}\n found: {:?}", - expected, actual - ); return Ok(SatisfiesResult::MismatchedOverrides(expected, actual)); } } @@ -790,7 +773,6 @@ impl Lock { let Some(root) = root else { // The package is not in the lockfile, so it can't be satisfied. - debug!("Workspace package `{root_name}` not found in lockfile"); return Ok(SatisfiesResult::MissingRoot(root_name.clone())); }; @@ -825,7 +807,6 @@ impl Lock { .get_or_build_wheel_metadata(&dist, HashPolicy::None) .await else { - debug!("Failed to get metadata for: {}", package.id); return Ok(SatisfiesResult::MissingMetadata( &package.id.name, &package.id.version, @@ -849,10 +830,6 @@ impl Lock { .collect::>()?; if expected != actual { - debug!( - "Mismatched `requires-dist` for {}:\n expected: {:?}\n found: {:?}", - package.id, expected, actual - ); return Ok(SatisfiesResult::MismatchedRequiresDist( &package.id.name, &package.id.version, @@ -895,10 +872,6 @@ impl Lock { .collect::>()?; if expected != actual { - debug!( - "Mismatched `requires-dev` for {}:\n expected: {:?}\n found: {:?}", - package.id, expected, actual - ); return Ok(SatisfiesResult::MismatchedDevDependencies( &package.id.name, &package.id.version,