vinzenz/hyperhive
vinzenz/hyperhive
Watch
0
0
Fork
You've already forked hyperhive
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

swarm-controller: report all assignees, not just one

Browse source
This commit is contained in:
damocles 2026-08-31 18:27:29 +02:00 • committed by mara
commit cc8e48a69b
1 changed files with 12 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

14
swarm-controller/src/forge.rs
View file

@ -56,7 +56,12 @@ pub struct IssueReportRow {
pub number: i64, pub number: i64,
pub title: String, pub title: String,
pub labels: Vec<String>, pub labels: Vec<String>,
pub assignee: Option<String>, /// All assignees, not just one — forgejo's `Issue::assignee` is a
/// legacy single-value field; `assignee`**s** (plural) is the real
/// multi-assignee list, which this repo's issues actually use (mara,
/// on review: "i thought you can assign multiple users, why just a
/// string?").
pub assignees: Vec<String>,
pub html_url: Option<String>, pub html_url: Option<String>,
/// True if any of this issue's own dependencies is still open — same /// True if any of this issue's own dependencies is still open — same
/// semantics `hive-forge issue dependency list` already uses (a closed /// semantics `hive-forge issue dependency list` already uses (a closed
@ -697,7 +702,12 @@ impl Client {
.into_iter() .into_iter()
.filter_map(|l| l.name) .filter_map(|l| l.name)
.collect(), .collect(),
assignee: issue.assignee.and_then(|a| a.login), assignees: issue
.assignees
.unwrap_or_default()
.into_iter()
.filter_map(|a| a.login)
.collect(),
html_url: issue.html_url.map(|u| u.to_string()), html_url: issue.html_url.map(|u| u.to_string()),
blocked, blocked,
depended_on_by_count: dependent_counts.get(&number).copied().unwrap_or(0), depended_on_by_count: dependent_counts.get(&number).copied().unwrap_or(0),