Remove Promise Polyfill

The Promise polyfill is not needed with the outlined techinal requirements for node or browser support. Closes Airtable#179 A follow up to: Airtable#191
bocoup · Jul 23, 2020 · d7554c8 · d7554c8
1 parent 5d3948d
commit d7554c8
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 14 deletions.
diff --git a/.eslintrc.json b/.eslintrc.json
@@ -108,5 +108,8 @@
         "space-unary-ops": "warn",
         "spaced-comment": "warn",
         "yoda": "warn"
+    },
+    "globals": {
+        "Promise": "writable"
     }
 }
diff --git a/lib/base.js b/lib/base.js
@@ -14,7 +14,6 @@ var HttpHeaders = require('./http_headers');
 var runAction = require('./run_action');
 var packageVersion = require('./package_version');
 var exponentialBackoffWithJitter = require('./exponential_backoff_with_jitter');
-var Promise = require('./promise');
 
 var userAgent = 'Airtable.js/' + packageVersion;
 

diff --git a/lib/callback_to_promise.js b/lib/callback_to_promise.js
@@ -1,7 +1,5 @@
 'use strict';
 
-var Promise = require('./promise');
-
 /**
  * Given a function fn that takes a callback as its last argument, returns
  * a new version of the function that takes the callback optionally. If

diff --git a/lib/promise.js b/lib/promise.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -16,7 +16,6 @@
   "dependencies": {
     "abort-controller": "^3.0.0",
     "abortcontroller-polyfill": "^1.4.0",
-    "es6-promise": "4.2.8",
     "lodash": "4.17.15",
     "node-fetch": "^2.6.0"
   },