diff options
| author | Kent Overstreet <kent.overstreet@linux.dev> | 2024-07-22 13:35:02 -0400 |
|---|---|---|
| committer | Kent Overstreet <kent.overstreet@linux.dev> | 2024-07-22 13:40:49 -0400 |
| commit | ab95495ce01b67b9be5db57c2c461db3358a5e22 (patch) | |
| tree | 57ad08b50aced10ac4112dd3e996f75b2bc61f46 | |
| parent | 39875687168166c080d20caaee633c68597598d2 (diff) | |
cgi.rs: Slim down output
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
| -rw-r--r-- | commit-filter | 4 | ||||
| -rw-r--r-- | src/bin/cgi.rs | 26 |
2 files changed, 16 insertions, 14 deletions
diff --git a/commit-filter b/commit-filter index c2abea9..fbd9c8b 100644 --- a/commit-filter +++ b/commit-filter @@ -29,7 +29,7 @@ maybe relocate the script block to within <head></head> <script> (function () { function get_row_status(el) { - return el.querySelector("td:nth-child(2)").textContent.trim() + return el.querySelector("td:nth-child(3)").textContent.trim() } function refresh_filters() { const shown_statuses = new Set() @@ -67,4 +67,4 @@ maybe relocate the script block to within <head></head> } }) })() -</script>
\ No newline at end of file +</script> diff --git a/src/bin/cgi.rs b/src/bin/cgi.rs index fd72ba8..08e65aa 100644 --- a/src/bin/cgi.rs +++ b/src/bin/cgi.rs @@ -218,18 +218,18 @@ fn last_good_line(results: &Vec<CommitResults>, test: &str) -> String { for (idx, result) in results.iter().map(|i| i.tests.get(test)).enumerate() { if let Some(result) = result { if result.status == TestStatus::Passed { - return format!("Passed {} commits ago", idx); + return format!("{}", idx); } if result.status != TestStatus::Failed { - return format!("Last {} commits failed", idx); + return format!(">= {}", idx); } } else { - return format!("Last {} commits failed", idx); + return format!(">= {}", idx); } } - return format!("Last {} commits failed", results.len()); + return format!(">= {}", results.len()); } fn ci_commit(ci: &Ci) -> cgi::Response { @@ -266,19 +266,21 @@ fn ci_commit(ci: &Ci) -> cgi::Response { for (name, result) in &first_commit.tests { writeln!(&mut out, "<tr class={}>", result.status.table_class()).unwrap(); - writeln!(&mut out, "<td> {} </td>", name).unwrap(); - writeln!(&mut out, "<td> {} </td>", result.status.to_str()).unwrap(); + writeln!(&mut out, "<td> {} </td>", name).unwrap(); writeln!(&mut out, "<td> {}s </td>", result.duration).unwrap(); - writeln!(&mut out, "<td> <a href=c/{}/{}/log.br> log </a> </td>", &first_commit.id, name).unwrap(); - writeln!(&mut out, "<td> <a href=c/{}/{}/full_log.br> full log </a> </td>", &first_commit.id, name).unwrap(); - writeln!(&mut out, "<td> <a href=c/{}/{}> output directory </a> </td>", &first_commit.id, name).unwrap(); - + writeln!(&mut out, "<td> {} </td>", result.status.to_str()).unwrap(); + writeln!(&mut out, "<td> {} </td>", last_good_line(&commits, name)).unwrap(); if let Some(branch) = &ci.branch { - writeln!(&mut out, "<td> <a href={}?branch={}&test=^{}$> git log </a> </td>", + writeln!(&mut out, "<td> <a href={}?branch={}&test=^{}$> log </a> </td>", ci.script_name, &branch, name).unwrap(); } + writeln!(&mut out, "<td> <a href=c/{}/{}/log.br> out </a> </td>", &first_commit.id, name).unwrap(); + writeln!(&mut out, "<td> <a href=c/{}/{}/full_log.br> full </a> </td>", &first_commit.id, name).unwrap(); + + /* We're not currently using this: + writeln!(&mut out, "<td> <a href=c/{}/{}> output directory </a> </td>", &first_commit.id, name).unwrap(); + */ - writeln!(&mut out, "<td> {} </td>", last_good_line(&commits, name)).unwrap(); writeln!(&mut out, "</tr>").unwrap(); } |