Skip to content

Commit e14f690

Browse files
committed
refactor(resolve): Provide full access to summary
1 parent c1fa840 commit e14f690

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/cargo/ops/cargo_update.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -757,30 +757,30 @@ fn report_latest(possibilities: &[IndexSummary], change: &PackageChange) -> Opti
757757
}
758758

759759
let version_req = package_id.version().to_caret_req();
760-
if let Some(version) = possibilities
760+
if let Some(summary) = possibilities
761761
.iter()
762762
.map(|s| s.as_summary())
763763
.filter(|s| package_id.version() != s.version() && version_req.matches(s.version()))
764-
.map(|s| s.version().clone())
765-
.max()
764+
.max_by_key(|s| s.version())
766765
{
767766
let warn = style::WARN;
767+
let version = summary.version();
768768
let report = format!(" {warn}(latest compatible: v{version}){warn:#}");
769769
return Some(report);
770770
}
771771

772-
if let Some(version) = possibilities
772+
if let Some(summary) = possibilities
773773
.iter()
774774
.map(|s| s.as_summary())
775775
.filter(|s| is_latest(s.version(), package_id.version()))
776-
.map(|s| s.version().clone())
777-
.max()
776+
.max_by_key(|s| s.version())
778777
{
779778
let warn = if change.is_transitive.unwrap_or(true) {
780779
Default::default()
781780
} else {
782781
style::WARN
783782
};
783+
let version = summary.version();
784784
let report = format!(" {warn}(latest: v{version}){warn:#}");
785785
return Some(report);
786786
}

0 commit comments

Comments
 (0)