From 23d8e89f03f232e049cd0886eaafa5a4f4d61b27 Mon Sep 17 00:00:00 2001 From: Luke Granger-Brown Date: Fri, 2 Apr 2021 19:43:32 +0000 Subject: [PATCH] chore(clbot): reformat messages Instead of putting the subject in quotes, we instead reorder the message to be of the format: CL/2768 applied by lukegb - chore(clbot): reformat messages - https://cl.tvl.fyi/2768 Change-Id: I77d19525c399396a91797f423bdfc92069d47f9f Reviewed-on: https://cl.tvl.fyi/c/depot/+/2768 Reviewed-by: glittershark Tested-by: BuildkiteCI --- fun/clbot/clbot.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fun/clbot/clbot.go b/fun/clbot/clbot.go index 691f43afb..bde8f1f99 100644 --- a/fun/clbot/clbot.go +++ b/fun/clbot/clbot.go @@ -232,13 +232,13 @@ func main() { continue } user := username(e.PatchSet) - parsedMsg = nopingAll(user, fmt.Sprintf("CL/%d: %q proposed by %s - %s", e.Change.Number, e.Change.Subject, user, patchSetURL(e.Change, e.PatchSet))) + parsedMsg = nopingAll(user, fmt.Sprintf("CL/%d proposed by %s - %s - %s", e.Change.Number, user, e.Change.Subject, patchSetURL(e.Change, e.PatchSet))) case *gerritevents.ChangeMerged: if e.Change.Project != *notifyRepo || !notifyBranches[e.Change.Branch] { continue } user := username(e.PatchSet) - parsedMsg = nopingAll(user, fmt.Sprintf("CL/%d: %q applied by %s - %s", e.Change.Number, e.Change.Subject, user, patchSetURL(e.Change, e.PatchSet))) + parsedMsg = nopingAll(user, fmt.Sprintf("CL/%d applied by %s - %s - %s", e.Change.Number, user, e.Change.Subject, patchSetURL(e.Change, e.PatchSet))) } if parsedMsg != "" { sendMsgChan <- parsedMsg