clippy

diary_app_api/src/errors.rs

@@ -1,9 +1,9 @@
 use anyhow::Error as AnyhowError;
 use handlebars::RenderError;
-use rweb::http::StatusCode;
 use log::error;
 use postgres_query::Error as PqError;
 use rweb::{
+    http::StatusCode,
     openapi::{
         ComponentDescriptor, ComponentOrInlineSchema, Entity, Response, ResponseEntity, Responses,
     },

diary_app_api/src/lib.rs

@@ -6,6 +6,7 @@
 #![allow(clippy::used_underscore_binding)]
 #![allow(clippy::unused_async)]
 #![allow(clippy::implicit_hasher)]
+#![allow(clippy::ignored_unit_patterns)]
 
 pub mod app;
 pub mod elements;

diary_app_bot/src/telegram_bot.rs

@@ -152,7 +152,7 @@ async fn telegram_worker(dapp: DiaryAppInterface) -> Result<(), Error> {
         let d = dapp.clone();
 
         match timeout(Duration::from_secs(3600), bot_handler(d)).await {
-            Err(_) | Ok(Ok(_)) => FAILURE_COUNT.reset()?,
+            Err(_) | Ok(Ok(())) => FAILURE_COUNT.reset()?,
             Ok(Err(_)) => FAILURE_COUNT.increment()?,
         }
     }

diary_app_lib/src/diary_app_interface.rs

@@ -307,11 +307,14 @@ impl DiaryAppInterface {
         let local = DateTimeWrapper::local_tz();
         let date_entry_map = DiaryCache::get_cache_entries(&self.pool)
             .await?
-            .try_fold(HashMap::new(), |mut acc, entry| async move {
-                let entry_date = entry.diary_datetime.to_timezone(local).date();
-                acc.entry(entry_date).or_insert_with(Vec::new).push(entry);
-                Ok(acc)
-            })
+            .try_fold(
+                HashMap::new(),
+                |mut acc: HashMap<Date, Vec<DiaryCache>>, entry| async move {
+                    let entry_date = entry.diary_datetime.to_timezone(local).date();
+                    acc.entry(entry_date).or_default().push(entry);
+                    Ok(acc)
+                },
+            )
             .await?;
 
         let futures: FuturesUnordered<_> = date_entry_map

diary_app_lib/src/local_interface.rs

@@ -51,7 +51,7 @@ impl LocalInterface {
         let year_map: BTreeMap<i32, Vec<_>> =
             date_list.into_iter().fold(BTreeMap::new(), |mut acc, d| {
                 let year = d.year();
-                acc.entry(year).or_insert_with(Vec::new).push(d);
+                acc.entry(year).or_default().push(d);
                 acc
             });
 

src/diary_app_rust.rs

@@ -5,7 +5,7 @@ async fn main() {
     env_logger::init();
 
     match DiaryAppOpts::process_args().await {
-        Ok(_) => {}
+        Ok(()) => {}
         Err(e) => {
             assert!(e.to_string().contains("Broken pipe"), "{}", e);
         }