Skip to content

Commit

Permalink
Replace the core-js promise polyfill
Browse files Browse the repository at this point in the history
The corejs polyfill was causing issues with @dojo/framework. Replace it
with promise-polyfill. Eventually, don't use a polyfill.

fixes #951
  • Loading branch information
jason0x43 committed Oct 19, 2018
1 parent d507b46 commit 91f61bc
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 2 deletions.
6 changes: 6 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -129,6 +129,7 @@
"lint-staged": "~7.2.0",
"pre-commit": "~1.2.2",
"prettier": "~1.13.7",
"promise-polyfill": "~8.1.0",
"sinon": "~2.3.6",
"source-map-support": "~0.5.6",
"style-loader": "~0.20.3",
Expand Down
4 changes: 2 additions & 2 deletions webpack.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -74,14 +74,14 @@ function getEntries() {
'url-search-params-polyfill',
'core-js/fn/map',
'core-js/fn/object/assign',
'core-js/fn/promise',
'promise-polyfill',
'./src/browser/intern.ts'
],
remote: [
'url-search-params-polyfill',
'core-js/fn/map',
'core-js/fn/object/assign',
'core-js/fn/promise',
'promise-polyfill',
'./src/browser/remote.ts'
],
config: './src/browser/config.ts'
Expand Down

0 comments on commit 91f61bc

Please sign in to comment.