Signed-off-by: Lars Wirzenius liw@liw.fi

May I suggest a different CLI language?

First of all, I think it would be a bit cool to have this command integrated into rad, by renaming it to rad-job.

Furthermore, partly to avoid repeating job, but also to align better with existing rad language, I’d like to propose the following subcommand changes.

Also to align with existing rad subcommands such as rad issue, list could be made the default command.

Finally, I have a small output change suggestion:

diff --git a/src/main.rs b/src/main.rs
index 30982a6..9d2e5b5 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -154,7 +154,7 @@ impl JobsList {
 
         line(&mut s, format!("count: {}", self.count));
         for shown in self.jobs.iter() {
-            line(&mut s, format!("job {}", shown.job_id));
+            line(&mut s, format!("job {} (oid {})", shown.job_id, shown.oid));
             for run in shown.runs.iter() {
                 line(&mut s, format!("  node {}", run.node_id));
                 for run2 in run.runs.iter() {

For any git user, the oid makes the job relatable to what they have in their commit history.

Apply suggestions from Richard Levitte

Changes:

• Introduce FindByCommit iterator
• Introduce display module
• Refactoring of CLI

Changes:

• Remove optional UUID for rad job run
• Rename Error to RadJobError
• Improve wording of doc strings

Changes:

• Re-attribution of commit