New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
introduce a SemanticSimilarity provider model #179640
Conversation
@@ -94,18 +94,21 @@ export function raceCancellationError<T>(promise: Promise<T>, token: Cancellatio | |||
} | |||
|
|||
/** | |||
* Returns as soon as one of the promises is resolved and cancels remaining promises | |||
* Returns as soon as one of the promises resolves or rejects and cancels remaining promises | |||
*/ | |||
export async function raceCancellablePromises<T>(cancellablePromises: CancelablePromise<T>[]): Promise<T> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is an improvement, thanks, but it is a behavior change so if you haven't already, make sure you take a look at the other places it's used and try to check whether that might be an issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah it's only used in one other place and I think based on the code it seems ok. @sandy081 FYI
No description provided.