chore(deps): bump yargs and typescript (#5629)

Author: farfromrefug

lib/common/file-system.ts

@@ -174,7 +174,7 @@ export class FileSystem implements IFileSystem {
 	}
 
 	public async futureFromEvent(eventEmitter: any, event: string): Promise<any> {
-		return new Promise((resolve, reject) => {
+		return new Promise<any>((resolve, reject) => {
 			eventEmitter.once(event, function () {
 				const args = _.toArray(arguments);
 
@@ -186,7 +186,7 @@ export class FileSystem implements IFileSystem {
 
 				switch (args.length) {
 					case 0:
-						resolve();
+						resolve(undefined);
 						break;
 					case 1:
 						resolve(args[0]);

lib/common/helpers.ts

@@ -198,7 +198,7 @@ export function settlePromises<T>(promises: Promise<T>[]): Promise<T[]> {
 		const length = promises.length;
 
 		if (!promises.length) {
-			resolve();
+			resolve(null);
 		}
 
 		_.forEach(promises, (currentPromise, index) => {
@@ -491,7 +491,7 @@ export async function getFuturesResults<T>(
 ): Promise<T[] | T[][]> {
 	const results = await Promise.all(promises);
 
-	return _(results).filter(predicate).flatten().value();
+	return _(results).filter(predicate).flatten().value() as T[] | T[][];
 }
 
 /**

lib/common/mobile/application-manager-base.ts

@@ -117,25 +117,25 @@ export abstract class ApplicationManagerBase
 		}
 	}
 
-	public abstract async installApplication(
+	public abstract installApplication(
 		packageFilePath: string,
 		appIdentifier?: string,
 		buildData?: IBuildData
 	): Promise<void>;
-	public abstract async uninstallApplication(
+	public abstract uninstallApplication(
 		appIdentifier: string
 	): Promise<void>;
-	public abstract async startApplication(
+	public abstract startApplication(
 		appData: Mobile.IApplicationData
 	): Promise<void>;
-	public abstract async stopApplication(
+	public abstract stopApplication(
 		appData: Mobile.IApplicationData
 	): Promise<void>;
-	public abstract async getInstalledApplications(): Promise<string[]>;
-	public abstract async getDebuggableApps(): Promise<
+	public abstract getInstalledApplications(): Promise<string[]>;
+	public abstract getDebuggableApps(): Promise<
 		Mobile.IDeviceApplicationInformation[]
 	>;
-	public abstract async getDebuggableAppViews(
+	public abstract getDebuggableAppViews(
 		appIdentifiers: string[]
 	): Promise<IDictionary<Mobile.IDebugWebViewInfo[]>>;
 

lib/common/mobile/ios/device/ios-device-operations.ts

@@ -84,7 +84,7 @@ export class IOSDeviceOperations
 			}
 
 			// We need this because we need to make sure that we have devices.
-			await new Promise((resolve, reject) => {
+			await new Promise<void>((resolve, reject) => {
 				let iterationsCount = 0;
 				const maxIterationsCount = 3;
 

lib/common/mobile/ios/ios-device-base.ts

@@ -57,7 +57,7 @@ export abstract class IOSDeviceBase implements Mobile.IiOSDevice {
 		}, `ios-debug-socket-${this.deviceInfo.identifier}-${appId}.lock`);
 	}
 
-	protected abstract async getDebugSocketCore(
+	protected abstract getDebugSocketCore(
 		appId: string
 	): Promise<net.Socket>;
 

lib/common/test/unit-tests/helpers.ts

@@ -99,7 +99,7 @@ describe("helpers", () => {
 			element: any,
 			passedChunkSize: number
 		) => {
-			return new Promise((resolve) =>
+			return new Promise<void>((resolve) =>
 				setImmediate(() => {
 					const remainingElements = _.difference(
 						initialDataValues,

lib/common/test/unit-tests/mobile/devices-service.ts

@@ -300,7 +300,7 @@ function resetDefaultSetInterval(): void {
 }
 
 async function assertOnNextTick(assertionFunction: Function): Promise<void> {
-	await new Promise((resolve) => {
+	await new Promise<void>((resolve) => {
 		setTimeout(() => {
 			assertionFunction();
 			resolve();

lib/options.ts

@@ -1,5 +1,6 @@
 import * as helpers from "./common/helpers";
-import * as yargs from "yargs";
+import * as yargs from 'yargs';
+import { hideBin } from 'yargs/helpers';
 import * as _ from "lodash";
 import {
 	IDictionary,
@@ -366,9 +367,9 @@ export class Options {
 			opts[this.getDashedOptionName(key)] = value;
 		});
 
-		const parsed = yargs(process.argv.slice(2)).version(false).help(false);
-		this.initialArgv = parsed.argv;
-		this.argv = parsed.options(<any>opts).argv;
+		const parsed = yargs(hideBin(process.argv)).version(false).help(false);
+		this.initialArgv = parsed.argv as any;
+		this.argv = parsed.options(<any>opts).argv as any;
 
 		// For backwards compatibility
 		// Previously profileDir had a default option and calling `this.$options.profileDir` always returned valid result.

lib/services/debug-service-base.ts

@@ -18,12 +18,12 @@ export abstract class DebugServiceBase
 
 	public abstract get platform(): string;
 
-	public abstract async debug(
+	public abstract debug(
 		debugData: IDebugData,
 		debugOptions: IDebugOptions
 	): Promise<IDebugResultInfo>;
 
-	public abstract async debugStop(): Promise<void>;
+	public abstract debugStop(): Promise<void>;
 
 	protected getCanExecuteAction(
 		deviceIdentifier: string

lib/services/livesync/android-device-livesync-service-base.ts

@@ -17,11 +17,11 @@ export abstract class AndroidDeviceLiveSyncServiceBase extends DeviceLiveSyncSer
 		super($platformsDataService, device);
 	}
 
-	public abstract async transferFilesOnDevice(
+	public abstract transferFilesOnDevice(
 		deviceAppData: Mobile.IDeviceAppData,
 		localToDevicePaths: Mobile.ILocalToDevicePathData[]
 	): Promise<void>;
-	public abstract async transferDirectoryOnDevice(
+	public abstract transferDirectoryOnDevice(
 		deviceAppData: Mobile.IDeviceAppData,
 		localToDevicePaths: Mobile.ILocalToDevicePathData[],
 		projectFilesPath: string