Add TypeScript linter check

dist/polyfill.js

@@ -5,8 +5,7 @@
 }(this, (function () { 'use strict';
 
 /**
- * @constructor
- * @extends {Promise}
+ * @this {Promise}
  */
 function finallyConstructor(callback) {
   var constructor = this.constructor;
@@ -39,6 +38,7 @@ function bind(fn, thisArg) {
 
 /**
  * @constructor
+ * @param {Function} fn
  */
 function Promise(fn) {
   if (!(this instanceof Promise))
@@ -173,6 +173,7 @@ Promise.prototype['catch'] = function(onRejected) {
 };
 
 Promise.prototype.then = function(onFulfilled, onRejected) {
+  // @ts-ignore
   var prom = new this.constructor(noop);
 
   handle(this, new Handler(onFulfilled, onRejected, prom));
@@ -259,7 +260,7 @@ Promise._unhandledRejectionFn = function _unhandledRejectionFn(err) {
   }
 };
 
-var global;
+/** @suppress {undefinedVars} */
 var globalNS = (function() {
   // the only reliable means to get the global object is
   // `Function('return this')()`
@@ -276,8 +277,8 @@ var globalNS = (function() {
   throw new Error('unable to locate global object');
 })();
 
-if (!globalNS.Promise) {
-  globalNS.Promise = Promise;
+if (!('Promise' in globalNS)) {
+  globalNS['Promise'] = Promise;
 } else if (!globalNS.Promise.prototype['finally']) {
   globalNS.Promise.prototype['finally'] = finallyConstructor;
 }

package.json

@@ -15,7 +15,8 @@
     "precommit": "lint-staged",
     "pretest": "npm run build:cjs",
     "closure": "google-closure-compiler --js=src/*.js --checks_only --module_resolution=node --compilation_level=ADVANCED",
-    "lint": "eslint src && npm run closure",
+    "typescript": "tsc --checkJS --allowJS --noEmit src/**",
+    "lint": "eslint src && npm run closure && npm run typescript",
     "test": "npm run lint && mocha && karma start --single-run",
     "prebuild": "rimraf lib dist",
     "build": "run-p build:**",
@@ -53,7 +54,8 @@
     "rimraf": "^2.6.2",
     "rollup": "^0.52.0",
     "rollup-plugin-uglify": "^2.0.1",
-    "sinon": "^1.17.2"
+    "sinon": "^1.17.2",
+    "typescript": "^2.9.2"
   },
   "keywords": [
     "promise",

src/finally.js

@@ -1,6 +1,5 @@
 /**
- * @constructor
- * @extends {Promise}
+ * @this {Promise}
  */
 function finallyConstructor(callback) {
   var constructor = this.constructor;

src/index.js

@@ -15,6 +15,7 @@ function bind(fn, thisArg) {
 
 /**
  * @constructor
+ * @param {Function} fn
  */
 function Promise(fn) {
   if (!(this instanceof Promise))
@@ -149,6 +150,7 @@ Promise.prototype['catch'] = function(onRejected) {
 };
 
 Promise.prototype.then = function(onFulfilled, onRejected) {
+  // @ts-ignore
   var prom = new this.constructor(noop);
 
   handle(this, new Handler(onFulfilled, onRejected, prom));

src/polyfill.js

@@ -1,7 +1,7 @@
 import Promise from './index';
 import promiseFinally from './finally';
 
-var global;
+/** @suppress {undefinedVars} */
 var globalNS = (function() {
   // the only reliable means to get the global object is
   // `Function('return this')()`
@@ -18,8 +18,8 @@ var globalNS = (function() {
   throw new Error('unable to locate global object');
 })();
 
-if (!globalNS.Promise) {
-  globalNS.Promise = Promise;
+if (!('Promise' in globalNS)) {
+  globalNS['Promise'] = Promise;
 } else if (!globalNS.Promise.prototype['finally']) {
   globalNS.Promise.prototype['finally'] = promiseFinally;
 }