Merge pull request #566 from nolanlawson/562

fix(promise.js): clarify nopromises with commment
localForage · Jan 25, 2017 · 02f0c83 · 02f0c83
2 parents 79d55df + 2d1a2ab
commit 02f0c83
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/utils/promise.js b/src/utils/promise.js
@@ -1,6 +1,8 @@
 // This is CommonJS because lie is an external dependency, so Rollup
 // can just ignore it.
-if (typeof Promise === 'undefined' && typeof require !== 'undefined') {
+if (typeof Promise === 'undefined') {
+    // In the "nopromises" build this will just throw if you don't have
+    // a global promise object, but it would throw anyway later.
     require('lie/polyfill');
 }
 export default Promise;