From e2893d1ee7ec950c27818f07c35748641974662a Mon Sep 17 00:00:00 2001 From: Joel Natividad <1980690+jqnatividad@users.noreply.github.com> Date: Tue, 5 Dec 2023 23:09:13 -0500 Subject: [PATCH] `clippy`:semicolon_if_nothing_returned warning: consider adding a `;` to the last statement for consistent formatting --> src/cmd/excel.rs:628:29 | 628 | ... record.push_field(if *b { "true" } else { "false" }) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: add a `;` here: `record.push_field(if *b { "true" } else { "false" });` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#semicolon_if_nothing_returned = note: `-W clippy::semicolon-if-nothing-returned` implied by `-W clippy::pedantic` = help: to override `-W clippy::pedantic` add `#[allow(clippy::semicolon_if_nothing_returned)]` --- src/cmd/excel.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cmd/excel.rs b/src/cmd/excel.rs index 2838e891e..a558f02da 100644 --- a/src/cmd/excel.rs +++ b/src/cmd/excel.rs @@ -625,7 +625,7 @@ pub fn run(argv: &[&str]) -> CliResult<()> { }, DataType::Error(ref e) => record.push_field(&format!("{e:?}")), DataType::Bool(ref b) => { - record.push_field(if *b { "true" } else { "false" }) + record.push_field(if *b { "true" } else { "false" }); }, DataType::DateTimeIso(ref dt) => record.push_field(dt), DataType::DurationIso(ref d) => record.push_field(d),