tree 7cd397e84835aef6169aca06f3aa48141b762a1d
parent 0a21cd2ddb432b32fbcf10f8e0decfda5021de4f
author Edwin Kempin <ekempin@google.com> 1570628038 +0200
committer David Pursehouse <dpursehouse@collab.net> 1570668525 +0900

Notify only owner if combined check state is updated to non-passing state

Reviewers are only interested in the combined check state when it turns
to a value that makes the change submittable. This is the case for the
states SUCCESSFUL and NOT_RELEVANT. Other combined check states are only
relevant for the change owner, hence for those an email notification
should only be sent to the change owner.

This is just the default for the notifications. Callers can always
override it by setting NotifyHandling in the REST input.

Rerunning a check cannot cause an update of the combined check state to
SUCCESSFUL or NOT_RELEVANT, hence there is no test and documentation for
this case

Signed-off-by: Edwin Kempin <ekempin@google.com>
Change-Id: I37d407d3336f42d5cd876cccd90ee115cf7fe41d