feat: allow signing commits

closes #73

src/bin/main.rs

@@ -7,7 +7,7 @@ use conventional_commit_parser::parse;
 use git2::Repository;
 use koji::answers::{get_extracted_answers, ExtractedAnswers};
 use koji::commit::{commit, write_commit_msg};
-use koji::config::Config;
+use koji::config::{Config, ConfigArgs};
 use koji::questions::create_prompt;
 
 #[derive(Parser, Debug)]
@@ -53,6 +53,12 @@ struct Args {
         help = "Enables issue prompt, which will append a reference to an issue in the commit body"
     )]
     issues: Option<bool>,
+
+    #[arg(
+        long,
+        help = "Sign the commit using the user's GPG key, if one is configured"
+    )]
+    sign: Option<bool>,
 }
 
 fn main() -> Result<()> {
@@ -64,6 +70,7 @@ fn main() -> Result<()> {
         emoji,
         hook,
         issues,
+        sign,
     } = Args::parse();
 
     // Find repo
@@ -84,7 +91,14 @@ fn main() -> Result<()> {
     }
 
     // Load config
-    let config = Config::new(config, autocomplete, breaking_changes, emoji, issues)?;
+    let config = Config::new(Some(ConfigArgs {
+        autocomplete,
+        breaking_changes,
+        emoji,
+        issues,
+        path: config,
+        sign,
+    }))?;
 
     // Get answers from interactive prompt
     let answers = create_prompt(&repo, commit_message, &config)?;
@@ -102,7 +116,14 @@ fn main() -> Result<()> {
     if hook {
         write_commit_msg(repo, commit_type, scope, summary, body, is_breaking_change)?;
     } else {
-        commit(commit_type, scope, summary, body, is_breaking_change)?;
+        commit(
+            commit_type,
+            scope,
+            summary,
+            body,
+            is_breaking_change,
+            config.sign,
+        )?;
     }
 
     Ok(())

src/lib/answers.rs

@@ -203,7 +203,7 @@ mod tests {
 
     #[test]
     fn test_get_summary() {
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         let commit_types = config.commit_types;
 
         let answer = Some(Answer::String("needed more badges".into()));
@@ -216,7 +216,7 @@ mod tests {
 
     #[test]
     fn test_get_summary_with_emoji() {
-        let config = Config::new(None, None, None, Some(true), None).unwrap();
+        let config = Config::new(None).unwrap();
         let commit_types = config.commit_types;
 
         let answer = Some(Answer::String("needed more badges".into()));
@@ -229,7 +229,7 @@ mod tests {
 
     #[test]
     fn test_get_summary_with_shortcode() {
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         let commit_types = config.commit_types;
 
         let answer = Some(Answer::String("needed more badges :badger:".into()));
@@ -341,7 +341,7 @@ mod tests {
             ),
         ]));
 
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         let extracted_answers = get_extracted_answers(&answers, &config).unwrap();
 
         assert_eq!(

src/lib/commit.rs

@@ -37,6 +37,7 @@ pub fn commit(
     summary: String,
     body: Option<String>,
     is_breaking_change: bool,
+    sign: bool,
 ) -> Result<()> {
     let cocogitto = CocoGitto::get()?;
 
@@ -47,7 +48,7 @@ pub fn commit(
         body,
         None,
         is_breaking_change,
-        false,
+        sign,
     )?;
 
     Ok(())

src/lib/config.rs

@@ -10,6 +10,7 @@ pub struct Config {
     pub commit_types: IndexMap<String, CommitType>,
     pub emoji: bool,
     pub issues: bool,
+    pub sign: bool,
 }
 
 #[derive(Clone, Debug, Deserialize, PartialEq, Eq)]
@@ -27,17 +28,31 @@ struct ConfigTOML {
     commit_types: Vec<CommitType>,
     pub emoji: Option<bool>,
     pub issues: Option<bool>,
+    pub sign: Option<bool>,
+}
+
+#[derive(Default)]
+pub struct ConfigArgs {
+    pub path: Option<String>,
+    pub autocomplete: Option<bool>,
+    pub breaking_changes: Option<bool>,
+    pub emoji: Option<bool>,
+    pub issues: Option<bool>,
+    pub sign: Option<bool>,
 }
 
 impl Config {
     /// Find a config and load it
-    pub fn new(
-        path: Option<String>,
-        autocomplete: Option<bool>,
-        breaking_changes: Option<bool>,
-        emoji: Option<bool>,
-        issues: Option<bool>,
-    ) -> Result<Self> {
+    pub fn new(args: Option<ConfigArgs>) -> Result<Self> {
+        let ConfigArgs {
+            path,
+            autocomplete,
+            breaking_changes,
+            emoji,
+            issues,
+            sign,
+        } = args.unwrap_or_default();
+
         // Get the default config
         let default_str = include_str!("../../meta/config/default.toml");
         let default_config: ConfigTOML =
@@ -92,6 +107,7 @@ impl Config {
             commit_types,
             emoji: emoji.unwrap_or(config.emoji.unwrap_or(false)),
             issues: issues.unwrap_or(config.issues.unwrap_or(true)),
+            sign: sign.unwrap_or(config.sign.unwrap_or(false)),
         })
     }
 }
@@ -102,25 +118,33 @@ mod tests {
 
     #[test]
     fn test_breaking_changes() {
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         assert_eq!(config.breaking_changes, true);
 
-        let config = Config::new(None, None, Some(false), None, None).unwrap();
+        let config = Config::new(Some(ConfigArgs {
+            breaking_changes: Some(false),
+            ..Default::default()
+        }))
+        .unwrap();
         assert_eq!(config.breaking_changes, false);
     }
 
     #[test]
     fn test_issues() {
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         assert_eq!(config.issues, true);
 
-        let config = Config::new(None, None, None, None, Some(false)).unwrap();
+        let config = Config::new(Some(ConfigArgs {
+            issues: Some(false),
+            ..Default::default()
+        }))
+        .unwrap();
         assert_eq!(config.issues, false);
     }
 
     #[test]
     fn test_commit_types() {
-        let config = Config::new(None, None, None, Some(true), None).unwrap();
+        let config = Config::new(None).unwrap();
         let commit_types = config.commit_types;
 
         assert_eq!(

src/lib/questions.rs

@@ -199,7 +199,7 @@ mod tests {
 
     #[test]
     fn test_format_commit_type_choice() {
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         let commit_types = config.commit_types;
 
         let choice =
@@ -213,7 +213,7 @@ mod tests {
 
     #[test]
     fn test_render_commit_type_choice_with_emoji() {
-        let config = Config::new(None, None, None, None, None).unwrap();
+        let config = Config::new(None).unwrap();
         let commit_types = config.commit_types;
 
         let choice =