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

package.jsonの整理 #55

Closed equal-l2 closed 3 years ago

equal-l2 commented 3 years ago
ghost commented 3 years ago

@equal-l2 --exit-child がどういう意味を持つのか私には分からないのですが、開発するときにはこれが必要だけど、本番ではちゃんとrespawnするという認識で良いですか?

equal-l2 commented 3 years ago

npm run dev --respawnで動かしてると普通はファイルに変更があったときにリロードしてくれるんですが、結構な頻度でリロードしてくれなくなる現象がありました。 以下のページにその対策が載っていたのを採用した形です。 https://stackoverflow.com/a/65623189

ghost commented 3 years ago

@equal-l2 とりあえず、dev用の話なので本番はちゃんと動作するし、違う奴使うという認識で良いですか

たぶん tsc -p だと思いますが

でdev用だとこういう問題が起きてるよ、っていう局所的な話ですよね?だったら私がmergeしますが

equal-l2 commented 3 years ago

本番の方に関しては触ってみてないのでなんとも言えませんね

ghost commented 3 years ago

@equal-l2 ecosystem.config.jsとかの雰囲気からしてたぶん npm run start が走ってるのでtscだと思います

そういう判断でmergeしても大丈夫ですか

equal-l2 commented 3 years ago

お願いします。

equal-l2 commented 3 years ago

ドキュメンテーション作るまでが完成とするとちょっと微妙な感じですが、ひとまず機能を実装するだけするなら来週中にはできるんじゃないかなと思ってます。

ghost commented 3 years ago

@equal-l2 後輩の育成や、技術的負債にならないために努力はしたいので、申し訳ないのですがおそらく比較的暇だと思う4月までにある程度の目途はつけておきたい、というお気持ちです