warning: returning the result of a `let` binding from a block --> src/model/table_path.rs:19:5 | 13 | / let absolute = if with_extension.is_absolute() { 14 | | with_extension 15 | | } else { 16 | | cwd.join(with_extension) 17 | | }; | |______- unnecessary `let` binding 18 | 19 | absolute | ^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.96.0/index.html#let_and_return = note: `#[warn(clippy::let_and_return)]` on by default help: return the expression directly | 13 ~ 14 | 15 ~ if with_extension.is_absolute() { 16 + with_extension 17 + } else { 18 + cwd.join(with_extension) 19 + } |

warning: consider using `sort_by_key` --> src/model/recent.rs:85:5 | 85 | items.sort_by(|a, b| b.opened_at.cmp(&a.opened_at)); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/rust-1.96.0/index.html#unnecessary_sort_by = note: `#[warn(clippy::unnecessary_sort_by)]` on by default help: try | 85 - items.sort_by(|a, b| b.opened_at.cmp(&a.opened_at)); 85 + items.sort_by_key(|b| std::cmp::Reverse(b.opened_at)); |