shufo/docker-phoenix

Dockerイメージにnpm がインストールされていない

Closed this issue · 2 comments

yuki-toida commented

https://github.com/shufo/docker-phoenix/blob/master/Dockerfile#L13

上記に nodejs-npm を追記すればインストールされます。

もし権限をいただけるのでしたら、プルリクさせていただきます。
ご確認よろしくお願いいたします。

shufo commented

@yuki-toida
Thank you for taking your time for the review!
紛らわしくて申し訳ないですがmasterの1.4.5等のイメージはproduction用に使いたいので今後nodejs消す予定です 🙇‍♂️ というのはproductionで必要なランタイム以外を入れているとimageが重くなってデプロイサイクルが長くなるのと問題切り分けが難しくなるためです.

開発に必要なランタイムはdevelopmentブランチでタグを付けている<version>-developmentのタグの方に入れてlocal developmentは<version>-developmentで行っていこうと思っています。

もし開発にnpmが必要な場合は<version>-developmentタグのイメージに既に入っているのでそれを使ってもらえればと思います。

example

docker run -it -w /code -v $(pwd):/code --entrypoint npm shufo/phoenix:1.4.5-development install
  • 👍1
yuki-toida commented

なるほど！了解しました！
では、その方針で対応しますね。
ご返信いただきありがとうございます。