tree f06af7cd6d175777c76f5a96c0f2c54318f10b82
parent ea23fefec2584c8eacaeb613b2a02170f8de930d
author Youssef Elghareeb <ghareeb@google.com> 1639397544 +0100
committer Youssef Elghareeb <ghareeb@google.com> 1639397626 +0100

CheckSubmitRule: Return Optional#empty() if all checkers are optional

If all configured checkers for a project are optional, it's better to
return an Optional#empty() instead of a record with status=OK.

This is also needed by submit requirements, since we convert results of
legacy submit rules to submit requirements. If we return an
Optional#empty(), the SR evaluator will just skip creating a submit
requirement result for it.

Google-Bug-Id: b/209752314
Change-Id: Ia96861ed4bd5b4644f79988e3a7a8d221d42a227