about summary refs log tree commit diff
diff options
context:
space:
mode:
authorGriffin Smith <grfn@gws.fyi>2021-12-14T02·36-0500
committerclbot <clbot@tvl.fyi>2021-12-14T15·42+0000
commit15e3afccef649db98f8ed59a94244c21a537a37a (patch)
tree9f9787d2f65dcc07b56eb0b7a61587917abb8d06
parentc7ebb34f8f105688a963944505b28d5a8a699d6f (diff)
fix(clbot): Noping autosubmitted message r/3239
Change-Id: I6b7e086d2f10751f2f8b8c90eded01edcbfabadb
Reviewed-on: https://cl.tvl.fyi/c/depot/+/4317
Tested-by: BuildkiteCI
Reviewed-by: tazjin <mail@tazj.in>
Autosubmit: grfn <grfn@gws.fyi>
-rw-r--r--fun/clbot/clbot.go6
1 files changed, 1 insertions, 5 deletions
diff --git a/fun/clbot/clbot.go b/fun/clbot/clbot.go
index ee250f706c8c..c259987a52d2 100644
--- a/fun/clbot/clbot.go
+++ b/fun/clbot/clbot.go
@@ -254,11 +254,7 @@ func main() {
 				url := patchSetURL(e.Change, e.PatchSet)
 
 				if submitter != owner && submitter == "clbot" {
-					// Ping CL author on IRC for autosubmitted messages, as this
-					// is not necessarily an action they would otherwise notice.
-					// We avoid pinging only for actions triggered by the same
-					// user.
-					parsedMsg = fmt.Sprintf("CL/%d by %s autosubmitted - %s - %s", e.Change.Number, owner, e.Change.Subject, url)
+					parsedMsg = nopingAll(owner, fmt.Sprintf("CL/%d by %s autosubmitted - %s - %s", e.Change.Number, owner, e.Change.Subject, url))
 				} else {
 					parsedMsg = nopingAll(owner, fmt.Sprintf("CL/%d applied by %s - %s - %s", e.Change.Number, owner, e.Change.Subject, url))
 				}