History

unify reporting style (#222)

Sebastian Thiel committed 5 years ago

Commit 99be2e11 parent ac910161 file · +7 −5Browse files

Unified · Split

cargo-smart-release/src/command/release/mod.rs+7−5
@@ -149,9 +149,11 @@ fn present_dependencies( · 148 unmodified lines
149 }149 }
150 150
151 let no_requested_crate_will_publish = num_refused == ctx.base.crate_names.len();151 let no_requested_crate_will_publish = num_refused == ctx.base.crate_names.len();
152 let no_crate_being_published_message = "No provided crate is actually eligible for publishing.";
152 if no_requested_crate_will_publish && !verbose {153 if no_requested_crate_will_publish && !verbose {
153 bail!(154 bail!(
154 "No provided crate is actually eligible for publishing. Use --verbose to see the release plan nonetheless."155 "{} Use --verbose to see the release plan nonetheless.",
156 no_crate_being_published_message
155 )157 )
156 }158 }
157 159
@@ -277,7 +279,7 @@ fn present_dependencies( · 119 unmodified lines
277 for (cause, deps_and_bumps) in affected_crates_by_cause {279 for (cause, deps_and_bumps) in affected_crates_by_cause {
278 let plural_s = (deps_and_bumps.len() != 1).then(|| "s").unwrap_or("");280 let plural_s = (deps_and_bumps.len() != 1).then(|| "s").unwrap_or("");
279 log::info!(281 log::info!(
280 "{} adjust {} manifest{} due to breaking change in '{}': {}",282 "{} adjust {} manifest version{} due to breaking change in '{}': {}",
281 will(dry_run),283 will(dry_run),
282 deps_and_bumps.len(),284 deps_and_bumps.len(),
283 plural_s,285 plural_s,
@@ -317,11 +319,11 @@ fn present_dependencies( · 33 unmodified lines
317 .then(|| "s")319 .then(|| "s")
318 .unwrap_or_default();320 .unwrap_or_default();
319 log::info!(321 log::info!(
320 "Manifest{} of {} package{} {} be adjusted as its direct dependencies see a version change: {}",322 "{} adjust version constraints in manifest{} of {} package{} as direct dependencies are changing: {}",
323 will(dry_run),
321 plural_s,324 plural_s,
322 crate_names_for_manifest_updates.len(),325 crate_names_for_manifest_updates.len(),
323 plural_s,326 plural_s,
324 will(dry_run),
325 crate_names_for_manifest_updates.join(", ")327 crate_names_for_manifest_updates.join(", ")
326 );328 );
327 }329 }
@@ -331,7 +333,7 @@ fn present_dependencies( · 3 unmodified lines
331 bail!("Aborting due to previous error(s)");333 bail!("Aborting due to previous error(s)");
332 } else {334 } else {
333 if no_requested_crate_will_publish {335 if no_requested_crate_will_publish {
334 bail!("No provided crate is actually eligible for publishing.")336 bail!(no_crate_being_published_message)
335 }337 }
336 Ok(())338 Ok(())
337 }339 }

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t