maxmustermann/TypeScript
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Add "implicit any" warning for shorthand ambient modules

Browse source
This commit is contained in:
Andy Hanson 2016-06-06 08:14:49 -07:00
parent ed6bfcdb08
commit f832488801
4 changed files with 21 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/compiler/checker.ts
View file

@ -16048,6 +16048,12 @@ namespace ts {
}
}
}
if (compilerOptions.noImplicitAny && !node.body) {
// Ambient shorthand module is an implicit any
reportImplicitAnyError(node, anyType);
}
if (node.body) {
checkSourceElement(node.body);
}

8
tests/baselines/reference/ambientShorthand_isImplicitAny.errors.txt Normal file
View file

@ -0,0 +1,8 @@
tests/cases/conformance/ambient/ambientShorthand_isImplicitAny.ts(1,16): error TS7005: Variable '"jquery"' implicitly has an 'any' type.
==== tests/cases/conformance/ambient/ambientShorthand_isImplicitAny.ts (1 errors) ====
declare module "jquery";
~~~~~~~~
!!! error TS7005: Variable '"jquery"' implicitly has an 'any' type.

5
tests/baselines/reference/ambientShorthand_isImplicitAny.js Normal file
View file

@ -0,0 +1,5 @@
//// [ambientShorthand_isImplicitAny.ts]
declare module "jquery";
//// [ambientShorthand_isImplicitAny.js]

2
tests/cases/conformance/ambient/ambientShorthand_isImplicitAny.ts Normal file
View file

@ -0,0 +1,2 @@
// @noImplicitAny: true
declare module "jquery";