Skip to content

Commit 2aa2a12

Browse files
committed
build: update to typescript 3.2
Updates to TypeScript 3.2 and fixes some compilation errors.
1 parent 177a433 commit 2aa2a12

17 files changed

+48
-24
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@
138138
"tsconfig-paths": "^2.3.0",
139139
"tslint": "^5.12.0",
140140
"tsutils": "^3.0.0",
141-
"typescript": "~3.1.1",
141+
"typescript": "~3.2.0",
142142
"uglify-js": "^2.8.14"
143143
}
144144
}

src/cdk/schematics/ng-update/upgrade-rules/attribute-selectors/attributeSelectorsStylesheetRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {IOptions, Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {AttributeSelectorUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -28,7 +30,7 @@ export class Rule extends Rules.AbstractRule {
2830
export class Walker extends ComponentWalker {
2931

3032
/** Change data that upgrades to the specified target version. */
31-
data = getUpgradeDataFromWalker(this, 'attributeSelectors');
33+
data: AttributeSelectorUpgradeData[] = getUpgradeDataFromWalker(this, 'attributeSelectors');
3234

3335
constructor(sourceFile: ts.SourceFile, options: IOptions) {
3436
super(sourceFile, options);

src/cdk/schematics/ng-update/upgrade-rules/attribute-selectors/attributeSelectorsTemplateRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {AttributeSelectorUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -27,7 +29,7 @@ export class Rule extends Rules.AbstractRule {
2729
export class Walker extends ComponentWalker {
2830

2931
/** Change data that upgrades to the specified target version. */
30-
data = getUpgradeDataFromWalker(this, 'attributeSelectors');
32+
data: AttributeSelectorUpgradeData[] = getUpgradeDataFromWalker(this, 'attributeSelectors');
3133

3234
visitInlineTemplate(node: ts.StringLiteralLike) {
3335
this._createReplacementsForContent(node, node.getText()).forEach(data => {

src/cdk/schematics/ng-update/upgrade-rules/class-names/classNamesIdentifierRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {RuleFailure, Rules, RuleWalker} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {ClassNameUpgradeData} from '../../data';
1214
import {
1315
isExportSpecifierNode,
1416
isImportSpecifierNode,
@@ -34,7 +36,7 @@ export class Rule extends Rules.AbstractRule {
3436
export class Walker extends RuleWalker {
3537

3638
/** Change data that upgrades to the specified target version. */
37-
data = getUpgradeDataFromWalker(this, 'classNames');
39+
data: ClassNameUpgradeData[] = getUpgradeDataFromWalker(this, 'classNames');
3840

3941
/**
4042
* List of identifier names that have been imported from `@angular/material` or `@angular/cdk`

src/cdk/schematics/ng-update/upgrade-rules/css-selectors/cssSelectorsStylesheetRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {IOptions, Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {CssSelectorUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -27,7 +29,7 @@ export class Rule extends Rules.AbstractRule {
2729
export class Walker extends ComponentWalker {
2830

2931
/** Change data that upgrades to the specified target version. */
30-
data = getUpgradeDataFromWalker(this, 'cssSelectors');
32+
data: CssSelectorUpgradeData[] = getUpgradeDataFromWalker(this, 'cssSelectors');
3133

3234
constructor(sourceFile: ts.SourceFile, options: IOptions) {
3335
super(sourceFile, options);

src/cdk/schematics/ng-update/upgrade-rules/css-selectors/cssSelectorsTemplateRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {CssSelectorUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -27,7 +29,7 @@ export class Rule extends Rules.AbstractRule {
2729
export class Walker extends ComponentWalker {
2830

2931
/** Change data that upgrades to the specified target version. */
30-
data = getUpgradeDataFromWalker(this, 'cssSelectors');
32+
data: CssSelectorUpgradeData[] = getUpgradeDataFromWalker(this, 'cssSelectors');
3133

3234
visitInlineTemplate(node: ts.StringLiteralLike) {
3335
this._createReplacementsForContent(node, node.getText()).forEach(data => {

src/cdk/schematics/ng-update/upgrade-rules/element-selectors/elementSelectorsStylesheetRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {IOptions, Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {ElementSelectorUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -27,7 +29,7 @@ export class Rule extends Rules.AbstractRule {
2729
export class Walker extends ComponentWalker {
2830

2931
/** Change data that upgrades to the specified target version. */
30-
data = getUpgradeDataFromWalker(this, 'elementSelectors');
32+
data: ElementSelectorUpgradeData[] = getUpgradeDataFromWalker(this, 'elementSelectors');
3133

3234
constructor(sourceFile: ts.SourceFile, options: IOptions) {
3335
super(sourceFile, options);

src/cdk/schematics/ng-update/upgrade-rules/element-selectors/elementSelectorsTemplateRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {ElementSelectorUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -27,7 +29,7 @@ export class Rule extends Rules.AbstractRule {
2729
export class Walker extends ComponentWalker {
2830

2931
/** Change data that upgrades to the specified target version. */
30-
data = getUpgradeDataFromWalker(this, 'elementSelectors');
32+
data: ElementSelectorUpgradeData[] = getUpgradeDataFromWalker(this, 'elementSelectors');
3133

3234
visitInlineTemplate(node: ts.StringLiteralLike) {
3335
this._createReplacementsForContent(node, node.getText()).forEach(data => {

src/cdk/schematics/ng-update/upgrade-rules/input-names/inputNamesStylesheetRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {IOptions, Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {InputNameUpgradeData} from '../../data';
1214
import {ExternalResource} from '../../tslint/component-file';
1315
import {ComponentWalker} from '../../tslint/component-walker';
1416
import {findAllSubstringIndices} from '../../typescript/literal';
@@ -32,7 +34,7 @@ export class Rule extends Rules.AbstractRule {
3234
export class Walker extends ComponentWalker {
3335

3436
/** Change data that upgrades to the specified target version. */
35-
data = getUpgradeDataFromWalker(this, 'inputNames');
37+
data: InputNameUpgradeData[] = getUpgradeDataFromWalker(this, 'inputNames');
3638

3739
constructor(sourceFile: ts.SourceFile, options: IOptions) {
3840
super(sourceFile, options);

src/cdk/schematics/ng-update/upgrade-rules/input-names/inputNamesTemplateRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {InputNameUpgradeData} from '../../data';
1214
import {findInputsOnElementWithAttr, findInputsOnElementWithTag} from '../../html-parsing/angular';
1315
import {ExternalResource} from '../../tslint/component-file';
1416
import {ComponentWalker} from '../../tslint/component-walker';
@@ -27,7 +29,7 @@ export class Rule extends Rules.AbstractRule {
2729
export class Walker extends ComponentWalker {
2830

2931
/** Change data that upgrades to the specified target version. */
30-
data = getUpgradeDataFromWalker(this, 'inputNames');
32+
data: InputNameUpgradeData[] = getUpgradeDataFromWalker(this, 'inputNames');
3133

3234
visitInlineTemplate(node: ts.StringLiteralLike) {
3335
this._createReplacementsForContent(node, node.getText()).forEach(data => {

src/cdk/schematics/ng-update/upgrade-rules/output-names/outputNamesTemplateRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {Replacement, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {OutputNameUpgradeData} from '../../data';
1214
import {
1315
findOutputsOnElementWithAttr,
1416
findOutputsOnElementWithTag,
@@ -30,7 +32,7 @@ export class Rule extends Rules.AbstractRule {
3032
export class Walker extends ComponentWalker {
3133

3234
/** Change data that upgrades to the specified target version. */
33-
data = getUpgradeDataFromWalker(this, 'outputNames');
35+
data: OutputNameUpgradeData[] = getUpgradeDataFromWalker(this, 'outputNames');
3436

3537
visitInlineTemplate(node: ts.StringLiteralLike) {
3638
this._createReplacementsForContent(node, node.getText()).forEach(data => {

src/cdk/schematics/ng-update/upgrade-rules/property-names/propertyNamesAccessRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {green, red} from 'chalk';
1010
import {ProgramAwareRuleWalker, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {PropertyNameUpgradeData} from '../../data';
1214
import {getUpgradeDataFromWalker} from '../../upgrade-data';
1315

1416
/**
@@ -24,7 +26,7 @@ export class Rule extends Rules.TypedRule {
2426
export class Walker extends ProgramAwareRuleWalker {
2527

2628
/** Change data that upgrades to the specified target version. */
27-
data = getUpgradeDataFromWalker(this, 'propertyNames');
29+
data: PropertyNameUpgradeData[] = getUpgradeDataFromWalker(this, 'propertyNames');
2830

2931
visitPropertyAccessExpression(node: ts.PropertyAccessExpression) {
3032
const hostType = this.getTypeChecker().getTypeAtLocation(node.expression);

src/cdk/schematics/ng-update/upgrade-rules/signature-check/methodCallsCheckRule.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99
import {bold} from 'chalk';
1010
import {ProgramAwareRuleWalker, RuleFailure, Rules} from 'tslint';
1111
import * as ts from 'typescript';
12+
13+
import {MethodCallUpgradeData} from '../../data';
1214
import {getUpgradeDataFromWalker} from '../../upgrade-data';
1315

1416
/**
@@ -24,7 +26,7 @@ export class Rule extends Rules.TypedRule {
2426
export class Walker extends ProgramAwareRuleWalker {
2527

2628
/** Change data that upgrades to the specified target version. */
27-
data = getUpgradeDataFromWalker(this, 'methodCallChecks');
29+
data: MethodCallUpgradeData[] = getUpgradeDataFromWalker(this, 'methodCallChecks');
2830

2931
visitCallExpression(node: ts.CallExpression) {
3032
if (ts.isPropertyAccessExpression(node.expression)) {

src/cdk/schematics/testing/test-case-setup.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,11 @@
99
import {getSystemPath, normalize} from '@angular-devkit/core';
1010
import {TempScopedNodeJsSyncHost} from '@angular-devkit/core/node/testing';
1111
import * as virtualFs from '@angular-devkit/core/src/virtual-fs/host';
12-
import {SchematicTestRunner} from '@angular-devkit/schematics/testing';
13-
import {mkdirpSync, readFileSync, writeFileSync, removeSync} from 'fs-extra';
12+
import {SchematicTestRunner, UnitTestTree} from '@angular-devkit/schematics/testing';
13+
import {mkdirpSync, readFileSync, removeSync, writeFileSync} from 'fs-extra';
1414
import {sync as globSync} from 'glob';
15-
import {dirname, join, basename, relative, sep} from 'path';
15+
import {basename, dirname, join, relative, sep} from 'path';
16+
1617
import {createTestApp, runPostScheduledTasks} from '../testing';
1718

1819
/** Suffix that indicates whether a given file is a test case input. */
@@ -32,7 +33,7 @@ export function readFileContent(filePath: string): string {
3233
*/
3334
export function createFileSystemTestApp(runner: SchematicTestRunner) {
3435
const tempFileSystemHost = new TempScopedNodeJsSyncHost();
35-
const appTree = createTestApp(runner, {name: 'cdk-testing'});
36+
const appTree: UnitTestTree = createTestApp(runner, {name: 'cdk-testing'});
3637
const tempPath = getSystemPath(tempFileSystemHost.root);
3738

3839
// Since the TSLint fix task expects all files to be present on the real file system, we

src/cdk/schematics/utils/ast.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,16 @@ import {getWorkspace} from '@schematics/angular/utility/config';
1515
import {findModuleFromOptions as internalFindModule} from '@schematics/angular/utility/find-module';
1616
import {getAppModulePath} from '@schematics/angular/utility/ng-ast-utils';
1717
import {getProjectMainFile} from './project-main-file';
18-
import {ts} from './version-agnostic-typescript';
18+
import {ts, typescript} from './version-agnostic-typescript';
1919

2020

2121
/** Reads file given path and returns TypeScript source file. */
22-
export function getSourceFile(host: Tree, path: string) {
22+
export function getSourceFile(host: Tree, path: string): typescript.SourceFile {
2323
const buffer = host.read(path);
2424
if (!buffer) {
2525
throw new SchematicsException(`Could not find file for path: ${path}`);
2626
}
27-
const content = buffer.toString();
28-
return ts.createSourceFile(path, content, ts.ScriptTarget.Latest, true);
27+
return ts.createSourceFile(path, buffer.toString(), ts.ScriptTarget.Latest, true);
2928
}
3029

3130
/** Import and add module to root app module. */

src/cdk/schematics/utils/version-agnostic-typescript.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,4 +36,4 @@ try {
3636
}
3737
}
3838

39-
export {ts};
39+
export {ts, typescript};

yarn.lock

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10499,12 +10499,12 @@ typedarray@^0.0.6:
1049910499
resolved "https://registry.yarnpkg.com/typedarray/-/typedarray-0.0.6.tgz#867ac74e3864187b1d3d47d996a78ec5c8830777"
1050010500
integrity sha1-hnrHTjhkGHsdPUfZlqeOxciDB3c=
1050110501

10502-
[email protected], typescript@~3.1.1, typescript@~3.1.6:
10502+
[email protected], typescript@~3.1.6:
1050310503
version "3.1.6"
1050410504
resolved "https://registry.yarnpkg.com/typescript/-/typescript-3.1.6.tgz#b6543a83cfc8c2befb3f4c8fba6896f5b0c9be68"
1050510505
integrity sha512-tDMYfVtvpb96msS1lDX9MEdHrW4yOuZ4Kdc4Him9oU796XldPYF/t2+uKoX0BBa0hXXwDlqYQbXY5Rzjzc5hBA==
1050610506

10507-
10507+
[email protected], typescript@~3.2.0:
1050810508
version "3.2.4"
1050910509
resolved "https://registry.yarnpkg.com/typescript/-/typescript-3.2.4.tgz#c585cb952912263d915b462726ce244ba510ef3d"
1051010510
integrity sha512-0RNDbSdEokBeEAkgNbxJ+BLwSManFy9TeXz8uW+48j/xhEXv1ePME60olyzw2XzUqUBNAYFeJadIqAgNqIACwg==

0 commit comments

Comments
 (0)