about summary refs log tree commit diff
path: root/fun
diff options
context:
space:
mode:
authorLuke Granger-Brown <git@lukegb.com>2021-04-02T19·43+0000
committerlukegb <lukegb@tvl.fyi>2021-04-02T20·05+0000
commit23d8e89f03f232e049cd0886eaafa5a4f4d61b27 (patch)
tree4014988fb68da6e209840b432b6cc60023169442 /fun
parent0456de373369ffa2b93884d48410d0b62fbc66ef (diff)
chore(clbot): reformat messages r/2410
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 <grfn@gws.fyi>
Tested-by: BuildkiteCI
Diffstat (limited to 'fun')
-rw-r--r--fun/clbot/clbot.go4
1 files changed, 2 insertions, 2 deletions
diff --git a/fun/clbot/clbot.go b/fun/clbot/clbot.go
index 691f43afb9c9..bde8f1f99466 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