Async functions now return bluebird promises, remove redundant Promise.resolve

This commit is contained in:
Luke Barnard 2017-07-13 17:51:14 +01:00
parent c47398e7fe
commit dcc4db53f9

View File

@ -58,9 +58,10 @@ export async function getCompletions(query: string, selection: SelectionRange, f
// single timeout to reject the Promise.all, reflect each one and once they've all // single timeout to reject the Promise.all, reflect each one and once they've all
// settled, filter for the fulfilled ones // settled, filter for the fulfilled ones
PROVIDERS.map((provider) => { PROVIDERS.map((provider) => {
// Convert to bluebird promise so that we can do a timeout return provider
const p = Promise.resolve(provider.getCompletions(query, selection, force)); .getCompletions(query, selection, force)
return p.timeout(PROVIDER_COMPLETION_TIMEOUT).reflect(); .timeout(PROVIDER_COMPLETION_TIMEOUT)
.reflect();
}), }),
); );