QuickInput: dangerous use of any

[bpasero]

Looking into the following method:

vscode/src/vs/base/parts/quickinput/browser/quickInput.ts

Line 1184 in 076a17f

pick<T extends IQuickPickItem, O extends IPickOptions<T>>(picks: Promise<QuickPickInput<T>[]> | QuickPickInput<T>[], options: O = <O>{}, token: CancellationToken = CancellationToken.None): Promise<O extends { canPickMany: true } ? T[] : T> {

There are a bunch of any usages that seem to hide the fact that the promise may return undefined. I suggest to remove the use of any in code as it bypasses the type checking of TypeScript entirely. If the type is really not known, rather use unknown instead and require casts at the place where it is being used.