Refactor existing uses of `noAwait` to use `void` operator instead. This is currently the recommended practice to call async functions without consuming the return values at Google. Change-Id: I34fbcee0c6ea01e7019f36f72073b016a4c57cf6 Reviewed-on: https://gerrit-review.googlesource.com/c/plugins/code-owners/+/433437 Reviewed-by: Dmitrii Filippov <dmfilippov@google.com> Tested-by: Zuul <zuul-63@gerritcodereview-ci.iam.gserviceaccount.com>
diff --git a/web/code-owners-service.ts b/web/code-owners-service.ts index 5f44337..9e38a6b 100644 --- a/web/code-owners-service.ts +++ b/web/code-owners-service.ts
@@ -56,8 +56,6 @@ maxConcurrentRequests?: number; } -function noAwait(_promise: Promise<unknown>) {} - /** * Service for the data layer used in the plugin UI. */ @@ -175,7 +173,7 @@ // new status - it is expected, that after several retry a status // for the newest patchset is returned this.reset(); - noAwait(this.prefetch()); + void this.prefetch() return await this.getStatus(); } return status;