diff options
| author | Luke Granger-Brown <git@lukegb.com> | 2021-04-02T19·43+0000 |
|---|---|---|
| committer | lukegb <lukegb@tvl.fyi> | 2021-04-02T20·05+0000 |
| commit | 23d8e89f03f232e049cd0886eaafa5a4f4d61b27 (patch) | |
| tree | 4014988fb68da6e209840b432b6cc60023169442 | |
| parent | 0456de373369ffa2b93884d48410d0b62fbc66ef (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
| -rw-r--r-- | fun/clbot/clbot.go | 4 |
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 |