[snowpatch] [PATCH] patchwork: Get rid of dummy URL for test results

Andrew Donnellan andrew.donnellan at au1.ibm.com
Thu Feb 8 14:25:55 AEDT 2018


At some point, Patchwork didn't like it when we submitted a Check without a
target_url attached, so we sent "no.url" as a default.

That appears to have changed, so get rid of the dummy URL.

Signed-off-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

---

applies on top of current patchwork series, completely untested
---
 src/patchwork.rs | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/patchwork.rs b/src/patchwork.rs
index 1aec0d4c7c8f..afbfb07774fa 100644
--- a/src/patchwork.rs
+++ b/src/patchwork.rs
@@ -146,9 +146,6 @@ pub struct TestResult {
 impl TestResult {
     pub fn as_json(&self) -> String {
         let mut result = self.clone();
-        if result.target_url.is_none() {
-            result.target_url = Some("http://no.url".to_string());
-        }
         if result.context.is_none() {
             result.context = Some(format!("{}-{}",
                                      env!("CARGO_PKG_NAME"),
-- 
2.11.0



More information about the snowpatch mailing list