-
Notifications
You must be signed in to change notification settings - Fork 27.1k
Description
@vsavkin accidentally turned off our type checking for the non-cjs tree.
I'm turning it back on, but it fails because of
upgrade/typings/jquery/jquery.d.ts (3186, 13): Subsequent variable declarations must have the same type. Variable '$' must be of type 'cssSelectorHelper', but here has type 'JQueryStatic'.
The reason is well-known and documented at DefinitelyTyped/DefinitelyTyped#2734 : Both JQuery and Selenium define a runtime variable in the global scope named $ so the typings files jquery.d.ts and angular-protractor.d.ts both define the symbol for the benefit of the type-checker, but with different types.
The right solution is to compile the modules/upgrade separately from the testing code that depends on protractor. This is a non-trivial change to the build.