Bump clechasseur/rs-clippy-check from 3 to 5 - #1

warning: method `get_current_target_info` is never used --> client/src/ui.rs:747:12 | 117 | impl App { | -------- method in this implementation ... 747 | pub fn get_current_target_info(&self) -> String { | ^^^^^^^^^^^^^^^^^^^^^^^

warning: variants `Busy` and `Away` are never constructed --> client/src/ui.rs:47:5 | 44 | pub enum Status { | ------ variants in this enum ... 47 | Busy, | ^^^^ 48 | Away, | ^^^^ | = note: `Status` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis

warning: field `current_user` is never read --> client/src/ui.rs:19:9 | 9 | pub struct App { | --- field in this struct ... 19 | pub current_user: String, | ^^^^^^^^^^^^ | = note: `App` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default

warning: unused variable: `target` --> client/src/ui.rs:498:65 | 498 | fn render_main_layout(&self, frame: &mut Frame, area: Rect, target: &str) { | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_target`

warning: unused variable: `target` --> client/src/ui.rs:484:75 | 484 | fn render_maximized_chat_layout(&self, frame: &mut Frame, area: Rect, target: &str) { | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_target` | = note: `#[warn(unused_variables)]` on by default

warning: use of deprecated method `ratatui::Frame::<'_>::set_cursor`: the method set_cursor_position indicates more clearly what about the cursor to set --> client/src/ui.rs:739:19 | 739 | frame.set_cursor( | ^^^^^^^^^^

warning: use of deprecated method `ratatui::Frame::<'_>::size`: use .area() as it's the more correct name --> client/src/ui.rs:470:26 | 470 | let size = frame.size(); | ^^^^ | = note: `#[warn(deprecated)]` on by default

warning: length comparison to zero --> client/src/main.rs:69:32 | 69 | ... if app.groups.len() > 0 { | ^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!app.groups.is_empty()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero

warning: length comparison to zero --> client/src/main.rs:60:32 | 60 | ... if app.contacts.len() > 0 { | ^^^^^^^^^^^^^^^^^^^^^^ help: using `!is_empty` is clearer and more explicit: `!app.contacts.is_empty()` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero = note: `#[warn(clippy::len_zero)]` on by default