tree cd64dd309e6830f537ac1d76645dab4742c63541
parent 1b388d270fd8fb4235b4a5317b0b16054c277114
author Dmitri Shuralyov <dmitshur@golang.org> 1561400119 -0400
committer Dmitri Shuralyov <dmitshur@golang.org> 1561400790 +0000

cmd/gopherbot: don't fight over NeedsDecision removal

This is a followup to CL 180925 to prevent gopherbot from fighting
people (too much) if they decide to override gopherbot's behavior
and re-add the NeedsDecision label. It's done by adding a check for
whether gopherbot has already taken the action and avoid repeating
it if so.

Since this action is removing a label, we can't just check for any
"labeled" event, as those are likely to happen for other reasons.
So add a more precise check for whether gopherbot has previously
removed the "NeedsDecision" label from the target issue.

Updates golang/go#31788
Updates golang/go#21312

Change-Id: Iaf4dd69a5bfd637694995ee60869f94362110a7d
Reviewed-on: https://go-review.googlesource.com/c/build/+/183624
Reviewed-by: Andrew Bonventre <andybons@golang.org>