Improvements to completion conditions
Long time no update! Lots of work happening behind the scenes — including many bug fixes — but finally here's some visible updates to announce related to custom review completion conditions.
First off, the result of your custom completion conditions will now automatically be supplemented by the output of the built-in default condition. Any properties missing from the object you return will be inherited from the default one instead. This will allow custom conditions to tweak things slightly without taking on the responsibility of maintaining all the completion logic.
Second, your condition output can now include a refreshTimestamp to indicate when the result should be refreshed (clamped to no less than 5 minutes from now). This is useful if your condition depends on the current time for its logic, so the outcome will update automatically at the right time rather than waiting for a user to trigger a refresh by, e.g., visiting the review.
There's also a new bit of data available to your condition: review.pullRequest.requestedTeams holds a list of teams whose review was requested for the PR. (Requested reviewers were already available via review.pullRequest.requestedReviewers.) Please add feedback to issue #608 if this looks promising but doesn't quite meet your needs.