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;
