Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove babel-plugin-transform-es2015-modules-commonjs dependency from package.json #266

Merged
merged 1 commit into from Mar 5, 2017

Conversation

ykzts
Copy link
Collaborator

@ykzts ykzts commented Mar 5, 2017

package.jsonからbabel-plugin-transform-es2015-modules-commonjsの依存を取り除く。

babel-plugin-transform-es2015-modules-commonjsbabel-preset-envが依存していて、他の箇所からは読み込まれていない。そのためpackage.jsonに依存の記述をする必要性はない。
またbabel-preset-envが指定するバージョンとずれが生じてしまったときに複数バージョンのものがインストールされることになってしまう。ダウンロード時間の無駄でしかなく、またストレージを圧迫することとなり無駄でしかない。

@ykzts ykzts added this to the 2.0.0 milestone Mar 5, 2017
@ykzts ykzts self-assigned this Mar 5, 2017
…om `package.json`

`package.json`から`babel-plugin-transform-es2015-modules-commonjs`の依存を取り除く。

`babel-plugin-transform-es2015-modules-commonjs`は`babel-preset-env`が依存していて、
他の箇所からは読み込まれていない。
そのため`package.json`に依存の記述をする必要性はない。
また`babel-preset-env`が指定するバージョンとずれが生じてしまったときに
複数バージョンのものがインストールされることになってしまう。
ダウンロード時間の無駄でしかなく、またストレージを圧迫することとなり無駄でしかない。
@coveralls
Copy link

Coverage Status

Coverage remained the same at 9.091% when pulling e0f707b on remove-dep into 651dbbe on master.

@ykzts ykzts merged commit bf56e37 into master Mar 5, 2017
@ykzts ykzts deleted the remove-dep branch March 5, 2017 03:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants