mab122
/
radicle-heartwood-lfs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

cli: Add state filters for `rad issue list` 

Allow filtering the issues by their state.
This commit is contained in:
Slack Coder 2023-05-27 18:35:28 -05:00 committed by Alexis Sellier
parent c25450d310
commit 02f5596c05
No known key found for this signature in database
1 changed files with 29 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
radicle-cli/src/commands/issue.rs
View File

@ -32,7 +32,7 @@ Usage
rad issue [<option>...] rad issue [<option>...]
rad issue delete <issue-id> [<option>...] rad issue delete <issue-id> [<option>...]
rad issue edit <issue-id> [<option>...] rad issue edit <issue-id> [<option>...]
rad issue list [--assigned <did>] [<option>...] rad issue list [--assigned <did>] [--closed | --open | --solved] [<option>...]
rad issue open [--title <title>] [--description <text>] [--tag <tag>] [<option>...] rad issue open [--title <title>] [--description <text>] [--tag <tag>] [<option>...]
rad issue react <issue-id> [--emoji <char>] [<option>...] rad issue react <issue-id> [--emoji <char>] [<option>...]
rad issue show <issue-id> [<option>...] rad issue show <issue-id> [<option>...]
@ -101,6 +101,7 @@ pub enum Operation {
}, },
List { List {
assigned: Option<Assigned>, assigned: Option<Assigned>,
state: Option<State>,
}, },
} }
@ -132,6 +133,19 @@ impl Args for Options {
Long("help") => { Long("help") => {
return Err(Error::Help.into()); return Err(Error::Help.into());
} }
Long("closed") if op.is_none() || op == Some(OperationName::List) => {
state = Some(State::Closed {
reason: CloseReason::Other,
});
}
Long("open") if op.is_none() || op == Some(OperationName::List) => {
state = Some(State::Open);
}
Long("solved") if op.is_none() || op == Some(OperationName::List) => {
state = Some(State::Closed {
reason: CloseReason::Solved,
});
}
Long("title") if op == Some(OperationName::Open) => { Long("title") if op == Some(OperationName::Open) => {
title = Some(parser.value()?.to_string_lossy().into()); title = Some(parser.value()?.to_string_lossy().into());
} }
@ -223,7 +237,7 @@ impl Args for Options {
OperationName::Delete => Operation::Delete { OperationName::Delete => Operation::Delete {
id: id.ok_or_else(|| anyhow!("an issue to remove must be provided"))?, id: id.ok_or_else(|| anyhow!("an issue to remove must be provided"))?,
}, },
OperationName::List => Operation::List { assigned }, OperationName::List => Operation::List { assigned, state },
}; };
Ok(( Ok((
@ -305,8 +319,8 @@ pub fn run(options: Options, ctx: impl term::Context) -> anyhow::Result<()> {
&signer, &signer,
)?; )?;
} }
Operation::List { assigned } => { Operation::List { assigned, state } => {
list(&issues, &assigned, &profile)?; list(&issues, &assigned, &state, &profile)?;
} }
Operation::Delete { id } => { Operation::Delete { id } => {
let id = id.resolve(&repo.backend)?; let id = id.resolve(&repo.backend)?;
@ -330,6 +344,7 @@ pub fn run(options: Options, ctx: impl term::Context) -> anyhow::Result<()> {
fn list( fn list(
issues: &Issues, issues: &Issues,
assigned: &Option<Assigned>, assigned: &Option<Assigned>,
state: &Option<State>,
profile: &profile::Profile, profile: &profile::Profile,
) -> anyhow::Result<()> { ) -> anyhow::Result<()> {
let me = *profile.id(); let me = *profile.id();
@ -351,8 +366,16 @@ fn list(
// Skip issues that failed to load. // Skip issues that failed to load.
continue; continue;
}; };
if Some(true) == assignee.map(|a| !issue.assigned().any(|v| v == Did::from(a))) {
continue; if let Some(a) = assignee {
if !issue.assigned().any(|v| v == Did::from(a)) {
continue;
}
}
if let Some(s) = state {
if s != issue.state() {
continue;
}
} }
all.push((id, issue)) all.push((id, issue))
} }