src/issue.rs

@@ -5,20 +5,16 @@ use std::str::FromStr;
 pub enum State {
     New,
     Backlog,
-    Blocked,
     InProgress,
     Done,
     WontDo,
 }
 
-pub type IssueHandle = String;
-
 #[derive(Debug, PartialEq)]
 pub struct Issue {
     pub title: String,
     pub description: Option<String>,
     pub state: State,
-    pub dependencies: Option<Vec<IssueHandle>>,
 }
 
 #[derive(Debug, thiserror::Error)]
@@ -37,8 +33,6 @@ impl FromStr for State {
             Ok(State::New)
         } else if s == "backlog" {
             Ok(State::Backlog)
-        } else if s == "blocked" {
-            Ok(State::Blocked)
         } else if s == "inprogress" {
             Ok(State::InProgress)
         } else if s == "done" {
@@ -56,7 +50,6 @@ impl Issue {
         let mut title: Option<String> = None;
         let mut description: Option<String> = None;
         let mut state = State::New; // default state, if not specified in the issue
-        let mut dependencies: Option<Vec<String>> = None;
 
         for direntry in dir.read_dir()? {
             if let Ok(direntry) = direntry {
@@ -68,12 +61,6 @@ impl Issue {
                 } else if file_name == "state" {
                     let state_string = std::fs::read_to_string(direntry.path())?;
                     state = State::from_str(state_string.trim())?;
-                } else if file_name == "dependencies" {
-                    let dep_strings = std::fs::read_to_string(direntry.path())?;
-                    let deps: Vec<IssueHandle> = dep_strings.lines().map(|dep|{IssueHandle::from(dep)}).collect();
-                    if deps.len() > 0 {
-                        dependencies = Some(deps);
-                    }
                 } else {
                     println!("ignoring unknown file in issue directory: {:?}", file_name);
                 }
@@ -88,7 +75,6 @@ impl Issue {
             title: title.unwrap(),
             description: description,
             state: state,
-            dependencies,
         })
     }
 }
@@ -105,7 +91,6 @@ mod tests {
             title: String::from("this is the title of my issue"),
             description: Some(String::from("This is the description of my issue.\nIt is multiple lines.\n* Arbitrary contents\n* But let's use markdown by convention\n")),
             state: State::New,
-            dependencies: None,
         };
         assert_eq!(issue, expected);
     }
@@ -118,7 +103,6 @@ mod tests {
             title: String::from("minimal"),
             description: None,
             state: State::InProgress,
-            dependencies: None,
         };
         assert_eq!(issue, expected);
     }

src/issues.rs

@@ -83,7 +83,6 @@ mod tests {
                 title: String::from("minimal"),
                 description: None,
                 state: crate::issue::State::InProgress,
-                dependencies: None,
             },
         );
         expected.add_issue(
@@ -92,7 +91,6 @@ mod tests {
                 title: String::from("this is the title of my issue"),
                 description: Some(String::from("This is the description of my issue.\nIt is multiple lines.\n* Arbitrary contents\n* But let's use markdown by convention\n")),
                 state: crate::issue::State::New,
-                dependencies: None,
             }
         );
         assert_eq!(issues, expected);
@@ -110,7 +108,6 @@ mod tests {
                 title: String::from("oh yeah we got titles"),
                 description: None,
                 state: crate::issue::State::Done,
-                dependencies: None,
             },
         );
         expected.add_issue(
@@ -121,50 +118,8 @@ mod tests {
                     "Lots of words\nthat don't say much\nbecause this is just\na test\n",
                 )),
                 state: crate::issue::State::WontDo,
-                dependencies: None,
-            },
-        );
-        assert_eq!(issues, expected);
-    }
-
-    #[test]
-    fn read_issues_0002() {
-        let issues_dir = std::path::Path::new("test/0002/");
-        let issues = Issues::new_from_dir(issues_dir).unwrap();
-
-        let mut expected = Issues::new();
-        expected.add_issue(
-            String::from("3fa5bfd93317ad25772680071d5ac3259cd2384f"),
-            crate::issue::Issue {
-                title: String::from("oh yeah we got titles"),
-                description: None,
-                state: crate::issue::State::Done,
-                dependencies: None,
-            },
-        );
-        expected.add_issue(
-            String::from("dd79c8cfb8beeacd0460429944b4ecbe95a31561"),
-            crate::issue::Issue {
-                title: String::from("issues out the wazoo"),
-                description: Some(String::from(
-                    "Lots of words\nthat don't say much\nbecause this is just\na test\n",
-                )),
-                state: crate::issue::State::WontDo,
-                dependencies: None,
-            },
-        );
-        expected.add_issue(
-            String::from("a85f81fc5f14cb5d4851dd445dc9744c7f16ccc7"),
-            crate::issue::Issue {
-                title: String::from("issue with dependencies"),
-                description: Some(String::from(
-                    "a test has begun\nfor dependencies we seek\nintertwining life",
-                )),
-                state: crate::issue::State::WontDo,
-                dependencies: Some(vec![crate::issue::IssueHandle::from("3fa5bfd93317ad25772680071d5ac3259cd2384f"), crate::issue::IssueHandle::from("dd79c8cfb8beeacd0460429944b4ecbe95a31561")]),
             },
         );
         assert_eq!(issues, expected);
     }
 }
-

test/0002/3fa5bfd93317ad25772680071d5ac3259cd2384f/state

@@ -1 +0,0 @@
-done

test/0002/3fa5bfd93317ad25772680071d5ac3259cd2384f/title

@@ -1 +0,0 @@
-oh yeah we got titles

test/0002/a85f81fc5f14cb5d4851dd445dc9744c7f16ccc7/dependencies

@@ -1,2 +0,0 @@
-3fa5bfd93317ad25772680071d5ac3259cd2384f
-dd79c8cfb8beeacd0460429944b4ecbe95a31561

test/0002/a85f81fc5f14cb5d4851dd445dc9744c7f16ccc7/description

@@ -1,3 +0,0 @@
-a test has begun
-for dependencies we seek
-intertwining life

test/0002/a85f81fc5f14cb5d4851dd445dc9744c7f16ccc7/state

@@ -1 +0,0 @@
-wontdo

test/0002/a85f81fc5f14cb5d4851dd445dc9744c7f16ccc7/title

@@ -1 +0,0 @@
-issue with dependencies

test/0002/config.toml

@@ -1 +0,0 @@
-states = [ "open", "closed" ]

test/0002/dd79c8cfb8beeacd0460429944b4ecbe95a31561/description

@@ -1,4 +0,0 @@
-Lots of words
-that don't say much
-because this is just
-a test

test/0002/dd79c8cfb8beeacd0460429944b4ecbe95a31561/state

@@ -1 +0,0 @@
-wontdo

test/0002/dd79c8cfb8beeacd0460429944b4ecbe95a31561/title

@@ -1 +0,0 @@
-issues out the wazoo