Skip to content

Commit 4c45c79

Browse files
devversionjelbourn
authored andcommitted
build: enable noUnusedParameters check in aot. (#8188)
1 parent 3b41005 commit 4c45c79

File tree

4 files changed

+4
-8
lines changed

4 files changed

+4
-8
lines changed

src/demo-app/tsconfig-aot.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@
77
// Needed for Moment.js since it doesn't have a default export.
88
"allowSyntheticDefaultImports": true,
99
"experimentalDecorators": true,
10-
// TODO(paul): Remove once Angular has been upgraded and supports noUnusedParameters in AOT.
11-
"noUnusedParameters": false,
10+
"noUnusedParameters": true,
1211
"strictNullChecks": true,
1312
"outDir": ".",
1413
"paths": {

src/e2e-app/tsconfig-build.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@
55
"declaration": true,
66
"emitDecoratorMetadata": true,
77
"experimentalDecorators": true,
8-
// TODO(paul): Remove once Angular has been upgraded and supports noUnusedParameters in AOT.
9-
"noUnusedParameters": false,
8+
"noUnusedParameters": true,
109
// Don't use strict nulls for the e2e-app because the material-examples are not
1110
// strict-null compliant.
1211
"strictNullChecks": false,

src/universal-app/tsconfig-build.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@
55
"declaration": true,
66
"stripInternal": false,
77
"experimentalDecorators": true,
8-
// TODO(crisbeto): disabled due to https://github.com/angular/angular/issues/17131
9-
"noUnusedParameters": false,
8+
"noUnusedParameters": true,
109
"strictNullChecks": true,
1110
"module": "commonjs",
1211
"moduleResolution": "node",

src/universal-app/tsconfig-prerender.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,7 @@
44
"declaration": false,
55
"stripInternal": false,
66
"experimentalDecorators": true,
7-
// TODO(paul): Remove once Angular has been upgraded and supports noUnusedParameters in AOT.
8-
"noUnusedParameters": false,
7+
"noUnusedParameters": true,
98
"strictNullChecks": true,
109
"module": "commonjs",
1110
"moduleResolution": "node",

0 commit comments

Comments
 (0)