fix: resolve clippy warnings for CI compliance

- Replace deprecated gix work_dir() with workdir()
- Rename CommitType::from_str to parse to avoid clippy
  should_implement_trait warning
- Add #[allow(dead_code)] for enum variants and parse() used
  only via ALL const and in tests

Author: Sephyi

src/domain/commit.rs

@@ -3,6 +3,7 @@
 // SPDX-License-Identifier: GPL-3.0-only
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
+#[allow(dead_code)]
 pub enum CommitType {
     Feat,
     Fix,
@@ -40,7 +41,8 @@ impl CommitType {
         }
     }
 
-    pub fn from_str(s: &str) -> Option<Self> {
+    #[allow(dead_code)]
+    pub fn parse(s: &str) -> Option<Self> {
         match s {
             "feat" => Some(Self::Feat),
             "fix" => Some(Self::Fix),

src/services/git.rs

@@ -17,7 +17,7 @@ impl GitService {
         let repo = gix::discover(".").map_err(|_| Error::NotAGitRepo)?;
 
         let work_dir = repo
-            .work_dir()
+            .workdir()
             .ok_or_else(|| Error::Git("Bare repository not supported".into()))?
             .to_path_buf();
 

tests/commit_type.rs

@@ -9,17 +9,17 @@ fn all_matches_enum_variants() {
     assert_eq!(CommitType::ALL.len(), 11);
     for s in CommitType::ALL {
         assert!(
-            CommitType::from_str(s).is_some(),
-            "ALL entry {:?} has no matching from_str result",
+            CommitType::parse(s).is_some(),
+            "ALL entry {:?} has no matching parse result",
             s
         );
     }
 }
 
 #[test]
-fn from_str_roundtrips() {
+fn parse_roundtrips() {
     for s in CommitType::ALL {
-        let ct = CommitType::from_str(s).unwrap();
+        let ct = CommitType::parse(s).unwrap();
         assert_eq!(
             ct.as_str(),
             *s,
@@ -31,10 +31,10 @@ fn from_str_roundtrips() {
 }
 
 #[test]
-fn from_str_rejects_invalid() {
+fn parse_rejects_invalid() {
     for invalid in &["yolo", "", "FEAT"] {
         assert!(
-            CommitType::from_str(invalid).is_none(),
+            CommitType::parse(invalid).is_none(),
             "expected None for {:?}, but got Some",
             invalid
         );
@@ -46,7 +46,7 @@ fn display_matches_as_str() {
     assert_eq!(format!("{}", CommitType::Feat), "feat");
 
     for s in CommitType::ALL {
-        let ct = CommitType::from_str(s).unwrap();
+        let ct = CommitType::parse(s).unwrap();
         assert_eq!(
             ct.to_string(),
             ct.as_str(),