fix: suppress closure warnings/errors (angular#861)

hmdhk · Aug 4, 2017 · deae751 · deae751
1 parent 0d12a27
commit deae751
Show file tree

Hide file tree

Showing 7 changed files with 27 additions and 3 deletions.
diff --git a/lib/browser/browser.ts b/lib/browser/browser.ts
@@ -5,6 +5,10 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+/**
+ * @fileoverview
+ * @suppress {missingRequire}
+ */
 
 import {findEventTasks} from '../common/events';
 import {patchTimer} from '../common/timers';
@@ -207,4 +211,4 @@ Zone.__load_patch('PromiseRejectionEvent', (global: any, Zone: ZoneType, api: _Z
 Zone.__load_patch('util', (global: any, Zone: ZoneType, api: _ZonePrivate) => {
   api.patchOnProperties = patchOnProperties;
   api.patchMethod = patchMethod;
-});
+});
diff --git a/lib/browser/property-descriptor.ts b/lib/browser/property-descriptor.ts
@@ -5,6 +5,10 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+/**
+ * @fileoverview
+ * @suppress {globalThis}
+ */
 
 import {isBrowser, isMix, isNode, patchClass, patchOnProperties, zoneSymbol} from '../common/utils';
 

diff --git a/lib/common/error-rewrite.ts b/lib/common/error-rewrite.ts
@@ -5,6 +5,10 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+/**
+ * @fileoverview
+ * @suppress {globalThis,undefinedVars}
+ */
 
 /**
  * Extend the Error with additional fields for rewritten stack frames

diff --git a/lib/common/events.ts b/lib/common/events.ts
@@ -5,6 +5,10 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+/**
+ * @fileoverview
+ * @suppress {missingRequire}
+ */
 import {attachOriginToPatched, zoneSymbol} from './utils';
 
 export const TRUE_STR = 'true';

diff --git a/lib/common/timers.ts b/lib/common/timers.ts
@@ -5,6 +5,10 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+/**
+ * @fileoverview
+ * @suppress {missingRequire}
+ */
 
 import {patchMethod} from './utils';
 

diff --git a/lib/common/utils.ts b/lib/common/utils.ts
@@ -8,7 +8,7 @@
 /**
  * Suppress closure compiler errors about unknown 'Zone' variable
  * @fileoverview
- * @suppress {undefinedVars,globalThis}
+ * @suppress {undefinedVars,globalThis,missingRequire}
  */
 
 // Hack since TypeScript isn't compiling this for a worker.
@@ -366,4 +366,4 @@ export function isIEOrEdge() {
     return ieOrEdge;
   } catch (error) {
   }
-}
+}
diff --git a/lib/zone-spec/wtf.ts b/lib/zone-spec/wtf.ts
@@ -5,6 +5,10 @@
  * Use of this source code is governed by an MIT-style license that can be
  * found in the LICENSE file at https://angular.io/license
  */
+/**
+ * @fileoverview
+ * @suppress {missingRequire}
+ */
 
 (function(global: any) {
   interface Wtf {