NIFCLOUD-mbaas / ncmb_js

ニフクラ mobile backend JavaScript SDK
https://mbaas.nifcloud.com
Other
28 stars 27 forks source link

Support babel-node build #220

Closed ichiwa closed 6 years ago

ichiwa commented 6 years ago

概要(Summary)

babel-node 側でも babel-polyfill を読み込んでいるので、二重読み込みのエラーになる。

動作確認手順(Step for Confirmation)

Create test.js like this;

var NCMB = require("ncmb");
$ npm install -g @babel/core @babel/node
$ npx babel-node test.js
ncmbsdk commented 6 years ago

@ichiwa issue ありがとうございます。こちらでも事象再現いたしました。 認識が不足しておりました。指摘ありがとうございます。

いただきました p-r ですが、お手数ですが2点ほど追加修正を依頼いたしたく存じます。

ichiwa commented 6 years ago

@ncmbsdk 返信ありがとうございます。 ご指摘の通り、idempotent-babel-polyfill を利用するように変更しました。 また、package-lock.json も削除しました。 再度レビューいただけると幸いです。