fafo/techtree
fafo/techtree
15
0
Fork 0
Code Issues 113 Pull requests Projects 2 Activity Actions

Add better panic messages where relevant

Browse source
This commit is contained in:
Rahix 2025-12-09 16:18:48 +01:00
parent d10b164b61
commit 83a610134c
2 changed files with 9 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
techtree-manager/src/main.rs
View file

@ -145,7 +145,7 @@ async fn run() -> anyhow::Result<()> {
} }
'issues: for issue in tree.iter_issues() { 'issues: for issue in tree.iter_issues() {
let subtree = tree.subtree_for_issue(issue).expect("TODO"); let subtree = tree.subtree_for_issue(issue).expect("issue from tree not found in tree");
let hash = subtree.stable_hash(); let hash = subtree.stable_hash();
let (bot_comment, _is_new) = issue::find_or_make_bot_comment(&ctx, issue) let (bot_comment, _is_new) = issue::find_or_make_bot_comment(&ctx, issue)

11
techtree-manager/src/tree.rs
View file

@ -165,8 +165,12 @@ impl Tree {
} }
pub fn add_dependency_by_issue_number(&mut self, dependant: u64, dependency: u64) { pub fn add_dependency_by_issue_number(&mut self, dependant: u64, dependency: u64) {
let a = self.find_element_by_issue_number(dependant).expect("TODO"); let a = self
let b = self.find_element_by_issue_number(dependency).expect("TODO"); .find_element_by_issue_number(dependant)
.expect("dependant element does not exist");
let b = self
.find_element_by_issue_number(dependency)
.expect("dependency element does not exist");
self.graph.add_edge(a, b, ()); self.graph.add_edge(a, b, ());
} }
@ -422,7 +426,8 @@ impl<'a> Subtree<'a> {
.collect(); .collect();
edges.sort(); edges.sort();
let json_data = serde_json::ser::to_string(&(nodes, edges, HASH_EPOCH)).expect("TODO"); let json_data = serde_json::ser::to_string(&(nodes, edges, HASH_EPOCH))
.expect("serialization for a stable hash failed");
sha256::digest(json_data) sha256::digest(json_data)
} }
} }