[snowpatch] [PATCH 1/2] main: Make apply_patch check description more useful

Andrew Donnellan andrew.donnellan at au1.ibm.com
Tue Mar 5 17:22:05 AEDT 2019


Reformat the apply_patch check description and include the SHA1 of the base
we applied patches on top of.

Signed-off-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>
---
 src/main.rs | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/src/main.rs b/src/main.rs
index 9bb195bbd891..5f60cc2293ea 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -244,10 +244,9 @@ fn test_patch(
                     state: TestState::Success,
                     description: Some(
                         format!(
-                            "{}/{}\n\n{}",
+                            "Successfully applied on branch {} ({})",
                             branch_name.to_string(),
-                            "apply_patch".to_string(),
-                            "Successfully applied".to_string()
+                            commit.id().to_string()
                         )
                         .to_string(),
                     ),
@@ -261,10 +260,9 @@ fn test_patch(
                     state: TestState::Warning,
                     description: Some(
                         format!(
-                            "{}/{}\n\n{}",
+                            "Failed to apply on branch {} ({})",
                             branch_name.to_string(),
-                            "apply_patch".to_string(),
-                            "Patch failed to apply".to_string()
+                            commit.id().to_string()
                         )
                         .to_string(),
                     ),
-- 
2.11.0



More information about the snowpatch mailing list