From 44ea384ee59fafb76882e1d6687bc4cbc6d61611 Mon Sep 17 00:00:00 2001
From: sigoden <sigoden@gmail.com>
Date: Mon, 30 Oct 2023 10:22:00 +0800
Subject: chore: remove unnecessary clippy macros

---
 src/config/message.rs | 1 -
 src/config/mod.rs     | 5 -----
 src/config/session.rs | 1 -
 3 files changed, 7 deletions(-)

(limited to 'src/config')

diff --git a/src/config/message.rs b/src/config/message.rs
index 3c23634..d5fcff1 100644
--- a/src/config/message.rs
+++ b/src/config/message.rs
@@ -17,7 +17,6 @@ impl Message {
     }
 }
 
-#[allow(clippy::module_name_repetitions)]
 #[derive(Debug, Clone, Deserialize, Serialize)]
 #[serde(rename_all = "snake_case")]
 pub enum MessageRole {
diff --git a/src/config/mod.rs b/src/config/mod.rs
index 42521cc..4dcc2c9 100644
--- a/src/config/mod.rs
+++ b/src/config/mod.rs
@@ -41,7 +41,6 @@ const SET_COMPLETIONS: [&str; 7] = [
     ".set dry_run false",
 ];
 
-#[allow(clippy::struct_excessive_bools)]
 #[derive(Debug, Clone, Deserialize)]
 #[serde(default)]
 pub struct Config {
@@ -105,7 +104,6 @@ impl Default for Config {
     }
 }
 
-#[allow(clippy::module_name_repetitions)]
 pub type SharedConfig = Arc<RwLock<Config>>;
 
 impl Config {
@@ -281,7 +279,6 @@ impl Config {
     }
 
     pub fn echo_messages(&self, content: &str) -> String {
-        #[allow(clippy::option_if_let_else)]
         if let Some(session) = self.session.as_ref() {
             session.echo_messages(content)
         } else if let Some(role) = self.role.as_ref() {
@@ -292,7 +289,6 @@ impl Config {
     }
 
     pub fn build_messages(&self, content: &str) -> Result<Vec<Message>> {
-        #[allow(clippy::option_if_let_else)]
         let messages = if let Some(session) = self.session.as_ref() {
             session.build_emssages(content)
         } else if let Some(role) = self.role.as_ref() {
@@ -552,7 +548,6 @@ impl Config {
         }
     }
 
-    #[allow(clippy::unused_self)] // TODO: do we need to take self here? it's not used in the fn
     fn open_message_file(&self) -> Result<File> {
         let path = Self::messages_file()?;
         ensure_parent_exists(&path)?;
diff --git a/src/config/session.rs b/src/config/session.rs
index 09fd1ff..5ecb1c2 100644
--- a/src/config/session.rs
+++ b/src/config/session.rs
@@ -110,7 +110,6 @@ impl Session {
         self.tokens = num_tokens_from_messages(&self.build_emssages(""));
     }
 
-    #[allow(clippy::unnecessary_wraps)]
     pub fn add_message(&mut self, input: &str, output: &str) -> Result<()> {
         let mut need_add_msg = true;
         if self.messages.is_empty() {
-- 
cgit v1.2.3