diff options
| author | Griffin Smith <grfn@gws.fyi> | 2021-12-14T02·36-0500 |
|---|---|---|
| committer | clbot <clbot@tvl.fyi> | 2021-12-14T15·42+0000 |
| commit | 15e3afccef649db98f8ed59a94244c21a537a37a (patch) | |
| tree | 9f9787d2f65dcc07b56eb0b7a61587917abb8d06 /fun | |
| parent | c7ebb34f8f105688a963944505b28d5a8a699d6f (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>
Diffstat (limited to 'fun')
| -rw-r--r-- | fun/clbot/clbot.go | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/fun/clbot/clbot.go b/fun/clbot/clbot.go index ee250f706c..c259987a52 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)) } |