mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(clbot): Noping autosubmitted message

Browse source 
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>
This commit is contained in:
Griffin Smith 2021-12-13 21:36:39 -05:00 committed by clbot
parent c7ebb34f8f
commit 15e3afccef
1 changed files with 1 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
fun/clbot/clbot.go
View file

@ -254,11 +254,7 @@ func main() {
url := patchSetURL(e.Change, e.PatchSet) url := patchSetURL(e.Change, e.PatchSet)
if submitter != owner && submitter == "clbot" { if submitter != owner && submitter == "clbot" {
// Ping CL author on IRC for autosubmitted messages, as this parsedMsg = nopingAll(owner, fmt.Sprintf("CL/%d by %s autosubmitted - %s - %s", e.Change.Number, owner, e.Change.Subject, url))
// 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)
} else { } else {
parsedMsg = nopingAll(owner, fmt.Sprintf("CL/%d applied by %s - %s - %s", e.Change.Number, owner, e.Change.Subject, url)) parsedMsg = nopingAll(owner, fmt.Sprintf("CL/%d applied by %s - %s - %s", e.Change.Number, owner, e.Change.Subject, url))
} }