Skip to content

Commit 0924f83

Browse files
committed
refactor(update): Consolidate latest message rendering
1 parent 9a170d7 commit 0924f83

File tree

1 file changed

+5
-12
lines changed

1 file changed

+5
-12
lines changed

src/cargo/ops/cargo_update.rs

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -498,10 +498,6 @@ fn print_lockfile_generation(
498498
status_locking(ws, num_pkgs)?;
499499

500500
for diff in diff {
501-
fn format_latest(version: semver::Version) -> String {
502-
let warn = style::WARN;
503-
format!(" {warn}(latest: v{version}){warn:#}")
504-
}
505501
let possibilities = if let Some(query) = diff.alternatives_query() {
506502
loop {
507503
match registry.query_vec(&query, QueryKind::Exact) {
@@ -555,10 +551,6 @@ fn print_lockfile_sync(
555551
status_locking(ws, num_pkgs)?;
556552

557553
for diff in diff {
558-
fn format_latest(version: semver::Version) -> String {
559-
let warn = style::WARN;
560-
format!(" {warn}(latest: v{version}){warn:#}")
561-
}
562554
let possibilities = if let Some(query) = diff.alternatives_query() {
563555
loop {
564556
match registry.query_vec(&query, QueryKind::Exact) {
@@ -650,10 +642,6 @@ fn print_lockfile_updates(
650642

651643
let mut unchanged_behind = 0;
652644
for diff in diff {
653-
fn format_latest(version: semver::Version) -> String {
654-
let warn = style::WARN;
655-
format!(" {warn}(latest: v{version}){warn:#}")
656-
}
657645
let possibilities = if let Some(query) = diff.alternatives_query() {
658646
loop {
659647
match registry.query_vec(&query, QueryKind::Exact) {
@@ -807,6 +795,11 @@ fn status_locking(ws: &Workspace<'_>, num_pkgs: usize) -> CargoResult<()> {
807795
Ok(())
808796
}
809797

798+
fn format_latest(version: semver::Version) -> String {
799+
let warn = style::WARN;
800+
format!(" {warn}(latest: v{version}){warn:#}")
801+
}
802+
810803
fn is_latest(candidate: &semver::Version, current: &semver::Version) -> bool {
811804
current < candidate
812805
// Only match pre-release if major.minor.patch are the same

0 commit comments

Comments
 (0)