add author and timestamp to Issue
This commit is contained in:
parent
37b7eb341f
commit
316ca3a901
3 changed files with 59 additions and 12 deletions
|
|
@ -106,6 +106,8 @@ fn handle_command(args: &Args, issues_dir: &std::path::Path) -> anyhow::Result<(
|
||||||
match issues.get_issue(issue_id) {
|
match issues.get_issue(issue_id) {
|
||||||
Some(issue) => {
|
Some(issue) => {
|
||||||
println!("issue {}", issue_id);
|
println!("issue {}", issue_id);
|
||||||
|
println!("author: {}", issue.author);
|
||||||
|
println!("timestamp: {}", issue.timestamp);
|
||||||
println!("state: {:?}", issue.state);
|
println!("state: {:?}", issue.state);
|
||||||
if let Some(dependencies) = &issue.dependencies {
|
if let Some(dependencies) = &issue.dependencies {
|
||||||
println!("dependencies: {:?}", dependencies);
|
println!("dependencies: {:?}", dependencies);
|
||||||
|
|
|
||||||
27
src/issue.rs
27
src/issue.rs
|
|
@ -20,9 +20,11 @@ pub type IssueHandle = String;
|
||||||
|
|
||||||
#[derive(Debug, PartialEq)]
|
#[derive(Debug, PartialEq)]
|
||||||
pub struct Issue {
|
pub struct Issue {
|
||||||
pub description: String,
|
pub author: String,
|
||||||
|
pub timestamp: chrono::DateTime<chrono::Local>,
|
||||||
pub state: State,
|
pub state: State,
|
||||||
pub dependencies: Option<Vec<IssueHandle>>,
|
pub dependencies: Option<Vec<IssueHandle>>,
|
||||||
|
pub description: String,
|
||||||
pub comments: Vec<crate::comment::Comment>,
|
pub comments: Vec<crate::comment::Comment>,
|
||||||
|
|
||||||
/// This is the directory that the issue lives in. Only used
|
/// This is the directory that the issue lives in. Only used
|
||||||
|
|
@ -119,10 +121,15 @@ impl Issue {
|
||||||
return Err(IssueError::IssueParseError);
|
return Err(IssueError::IssueParseError);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
let author = crate::git::git_log_oldest_author(dir)?;
|
||||||
|
let timestamp = crate::git::git_log_oldest_timestamp(dir)?;
|
||||||
|
|
||||||
Ok(Self {
|
Ok(Self {
|
||||||
description: description.unwrap(),
|
author,
|
||||||
|
timestamp,
|
||||||
state: state,
|
state: state,
|
||||||
dependencies,
|
dependencies,
|
||||||
|
description: description.unwrap(),
|
||||||
comments,
|
comments,
|
||||||
dir: std::path::PathBuf::from(dir),
|
dir: std::path::PathBuf::from(dir),
|
||||||
})
|
})
|
||||||
|
|
@ -169,9 +176,11 @@ impl Issue {
|
||||||
issue_dir.push(&format!("{:032x}", rnd));
|
issue_dir.push(&format!("{:032x}", rnd));
|
||||||
std::fs::create_dir(&issue_dir)?;
|
std::fs::create_dir(&issue_dir)?;
|
||||||
Ok(Self {
|
Ok(Self {
|
||||||
description: String::from(""), // FIXME: kind of bogus to use the empty string as None
|
author: String::from(""),
|
||||||
|
timestamp: chrono::Local::now(),
|
||||||
state: State::New,
|
state: State::New,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from(""), // FIXME: kind of bogus to use the empty string as None
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir: issue_dir,
|
dir: issue_dir,
|
||||||
})
|
})
|
||||||
|
|
@ -245,9 +254,13 @@ mod tests {
|
||||||
let issue_dir = std::path::Path::new("test/0000/3943fc5c173fdf41c0a22251593cd476d96e6c9f/");
|
let issue_dir = std::path::Path::new("test/0000/3943fc5c173fdf41c0a22251593cd476d96e6c9f/");
|
||||||
let issue = Issue::new_from_dir(issue_dir).unwrap();
|
let issue = Issue::new_from_dir(issue_dir).unwrap();
|
||||||
let expected = Issue {
|
let expected = Issue {
|
||||||
description: String::from("this is the title of my issue\n\nThis is the description of my issue.\nIt is multiple lines.\n* Arbitrary contents\n* But let's use markdown by convention\n"),
|
author: String::from("Sebastian Kuzminsky <seb@highlab.com>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-03T12:14:26-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: State::New,
|
state: State::New,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("this is the title of my issue\n\nThis is the description of my issue.\nIt is multiple lines.\n* Arbitrary contents\n* But let's use markdown by convention\n"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir: std::path::PathBuf::from(issue_dir),
|
dir: std::path::PathBuf::from(issue_dir),
|
||||||
};
|
};
|
||||||
|
|
@ -259,9 +272,13 @@ mod tests {
|
||||||
let issue_dir = std::path::Path::new("test/0000/7792b063eef6d33e7da5dc1856750c149ba678c6/");
|
let issue_dir = std::path::Path::new("test/0000/7792b063eef6d33e7da5dc1856750c149ba678c6/");
|
||||||
let issue = Issue::new_from_dir(issue_dir).unwrap();
|
let issue = Issue::new_from_dir(issue_dir).unwrap();
|
||||||
let expected = Issue {
|
let expected = Issue {
|
||||||
description: String::from("minimal"),
|
author: String::from("Sebastian Kuzminsky <seb@highlab.com>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-03T12:14:26-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: State::InProgress,
|
state: State::InProgress,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("minimal"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir: std::path::PathBuf::from(issue_dir),
|
dir: std::path::PathBuf::from(issue_dir),
|
||||||
};
|
};
|
||||||
|
|
|
||||||
|
|
@ -96,9 +96,13 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("minimal"),
|
author: String::from("Sebastian Kuzminsky <seb@highlab.com>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-03T12:14:26-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::InProgress,
|
state: crate::issue::State::InProgress,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("minimal"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir,
|
dir,
|
||||||
},
|
},
|
||||||
|
|
@ -110,9 +114,13 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("this is the title of my issue\n\nThis is the description of my issue.\nIt is multiple lines.\n* Arbitrary contents\n* But let's use markdown by convention\n"),
|
author: String::from("Sebastian Kuzminsky <seb@highlab.com>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-03T12:14:26-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::New,
|
state: crate::issue::State::New,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("this is the title of my issue\n\nThis is the description of my issue.\nIt is multiple lines.\n* Arbitrary contents\n* But let's use markdown by convention\n"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir,
|
dir,
|
||||||
}
|
}
|
||||||
|
|
@ -133,9 +141,13 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("oh yeah we got titles"),
|
author: String::from("Sebastian Kuzminsky <seb@highlab.com>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-03T11:59:44-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::Done,
|
state: crate::issue::State::Done,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("oh yeah we got titles"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir,
|
dir,
|
||||||
},
|
},
|
||||||
|
|
@ -161,9 +173,13 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("issues out the wazoo\n\nLots of words\nthat don't say much\nbecause this is just\na test\n"),
|
author: String::from("Sebastian Kuzminsky <seb@highlab.com>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-03T11:59:44-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::WontDo,
|
state: crate::issue::State::WontDo,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("issues out the wazoo\n\nLots of words\nthat don't say much\nbecause this is just\na test\n"),
|
||||||
comments: expected_comments,
|
comments: expected_comments,
|
||||||
dir,
|
dir,
|
||||||
},
|
},
|
||||||
|
|
@ -184,9 +200,13 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("oh yeah we got titles\n"),
|
author: String::from("sigil-03 <sigil@glyphs.tech>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-05T13:55:49-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::Done,
|
state: crate::issue::State::Done,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("oh yeah we got titles\n"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir,
|
dir,
|
||||||
},
|
},
|
||||||
|
|
@ -198,9 +218,13 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("issues out the wazoo\n\nLots of words\nthat don't say much\nbecause this is just\na test\n"),
|
author: String::from("sigil-03 <sigil@glyphs.tech>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-05T13:55:49-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::WontDo,
|
state: crate::issue::State::WontDo,
|
||||||
dependencies: None,
|
dependencies: None,
|
||||||
|
description: String::from("issues out the wazoo\n\nLots of words\nthat don't say much\nbecause this is just\na test\n"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir,
|
dir,
|
||||||
},
|
},
|
||||||
|
|
@ -212,12 +236,16 @@ mod tests {
|
||||||
expected.add_issue(
|
expected.add_issue(
|
||||||
uuid,
|
uuid,
|
||||||
crate::issue::Issue {
|
crate::issue::Issue {
|
||||||
description: String::from("issue with dependencies\n\na test has begun\nfor dependencies we seek\nintertwining life"),
|
author: String::from("sigil-03 <sigil@glyphs.tech>"),
|
||||||
|
timestamp: chrono::DateTime::parse_from_rfc3339("2025-07-05T13:55:49-06:00")
|
||||||
|
.unwrap()
|
||||||
|
.with_timezone(&chrono::Local),
|
||||||
state: crate::issue::State::WontDo,
|
state: crate::issue::State::WontDo,
|
||||||
dependencies: Some(vec![
|
dependencies: Some(vec![
|
||||||
crate::issue::IssueHandle::from("3fa5bfd93317ad25772680071d5ac3259cd2384f"),
|
crate::issue::IssueHandle::from("3fa5bfd93317ad25772680071d5ac3259cd2384f"),
|
||||||
crate::issue::IssueHandle::from("dd79c8cfb8beeacd0460429944b4ecbe95a31561"),
|
crate::issue::IssueHandle::from("dd79c8cfb8beeacd0460429944b4ecbe95a31561"),
|
||||||
]),
|
]),
|
||||||
|
description: String::from("issue with dependencies\n\na test has begun\nfor dependencies we seek\nintertwining life"),
|
||||||
comments: Vec::<crate::comment::Comment>::new(),
|
comments: Vec::<crate::comment::Comment>::new(),
|
||||||
dir,
|
dir,
|
||||||
},
|
},
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue