Remove isOptional from IPackage

Author: JoeRobich

src/lsptoolshost/extensions/builtInComponents.ts

@@ -3,6 +3,7 @@
  *  Licensed under the MIT License. See License.txt in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
+import * as vscode from 'vscode';
 import * as fs from 'fs';
 import * as path from 'path';
 import { LanguageServerOptions } from '../../shared/options';
@@ -49,7 +50,7 @@ export const componentInfo: { [key: string]: ComponentInfo } = {
 export function getComponentPaths(
     componentName: string,
     options: LanguageServerOptions | undefined,
-    channel?: { warn: (message: string) => void }
+    channel?: vscode.LogOutputChannel
 ): string[] {
     const component = componentInfo[componentName];
     const baseFolder = getComponentFolderPath(component, options);

src/packageManager/IPackage.ts

@@ -13,5 +13,4 @@ export interface IPackage {
     platformId?: string;
     integrity?: string;
     isFramework?: boolean;
-    isOptional?: boolean;
 }

src/packageManager/absolutePathPackage.ts

@@ -20,8 +20,7 @@ export class AbsolutePathPackage implements IPackage {
         public fallbackUrl?: string,
         public platformId?: string,
         public integrity?: string,
-        public isFramework?: boolean,
-        public isOptional?: boolean
+        public isFramework?: boolean
     ) {}
 
     public static getAbsolutePathPackage(pkg: Package, extensionPath: string) {
@@ -37,8 +36,7 @@ export class AbsolutePathPackage implements IPackage {
             pkg.fallbackUrl,
             pkg.platformId,
             pkg.integrity,
-            pkg.isFramework,
-            pkg.isOptional
+            pkg.isFramework
         );
     }
 }