search

su-its / ams-backend

:briefcase: (This repository is no longer maintained) The backend server of our Access-management-system.
MIT License
0 stars 0 forks source link

node_moduleがインストールされない不具合を修正 #10

Closed ghost closed 3 years ago

ghost commented 3 years ago

ビルド時にnpm installしてたので、シェルスクリプトにしました

h-takeyeah commented 3 years ago

@Stroheim001 この変更によって何がどうなるのでしょうか。今まで新しくモジュールを追加したときは手作業でnpm installしていましたが、その辺が楽になるということですか？

ghost commented 3 years ago

@h-takeyeah 完全にDocker上で完結することを前提に組みましたのでかなり速度が遅くなるとは思いますが、そのための処置です。ほかの方法もありますが、結局build後にnpm installしなければならないのには変わりはないので、たぶんこんな感じの雰囲気になると思います。自分の知見不足なだけかもしれません。 ただ、windowsにnodejsを入れたくないとかそういう人向けにはなってしまっているので、いらないと思う人には邪魔だと思います。

h-takeyeah commented 3 years ago

ありがとうございました。マージしました