Closed michelle192837 closed 5 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: michelle192837
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Moved the query code into a separate package so it can be used in config validation without a circular dependency.
Nit: if you did this first in a separate commit, (NOP change) then that would have made it a touch easier to review the business logic changes.
/lgtm /hold ?
Noted for future, thanks! ^^
/hold cancel
If searching for a target (as opposed to labels or other identifiers), we want to search for those results using a difference method (SearchConfiguredTargets, rather than SearchInvocations). Add the search method and use it if the query contains a target atom (and update the Prow label accordingly for this search).
Moved the query code into a separate package so it can be used in config validation without a circular dependency.