naokazuterada / middleman-template

@karappo flavored middleman template
https://middleman-tmpl.vercel.app/
0 stars 1 forks source link

Remove deprecated packages #29

Closed tryptophannn closed 3 years ago

tryptophannn commented 3 years ago

目的

deprecatedなnpm packageの削除

1. 下記の非推奨のnpm packageの削除

2. babel-polyfillの代替案として、core-jsを使う方法を採用しました。

naokazuterada commented 3 years ago

@tryptophannn ありがとうございます! core-jsは、dependenciesに入れる必要があるのでしょうか?(devDependensiesでは不可?)

tryptophannn commented 3 years ago

@naokazuterada 参考記事にある通りに実装したのですが、公式でもdependenciesにinstallしていました。理由までは調査しないと現時点では不明です。

naokazuterada commented 3 years ago

@tryptophannn おそらくdevDependenciesで動くと思うのでそちらに移動して、build成功するかみてもらって良いでしょうか?元々devDependenciesにあったライブラリを消して、あえてdependenciesに入れるのであれば理由が分かっていないと気持ちが悪いので。devDependenciesで動かなくなってしまうのであれば、それを確認した上でdependenciesに追加しましょう。

また、公式のドキュメントはプロジェクト内のコードから直接core-jsをimportするような場合を想定しているためにdependenciesに入れているだけな気がします。

tryptophannn commented 3 years ago

@naokazuterada ご確認ありがとうございます。

devDependenciesで動くと思うのでそちらに移動して、build成功するかみてもらって良いでしょうか?

下記のコミットで対応しました。問題なく、buildできています。 d361b57

naokazuterada commented 3 years ago

@tryptophannn おそくなりました…。ありがとうございます!