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

dockerfileを書いていきます #5

Merged
merged 2 commits into from
Jul 7, 2024
Merged

Conversation

Holoscopecheck
Copy link
Collaborator

No description provided.

@Holoscopecheck Holoscopecheck linked an issue Jul 2, 2024 that may be closed by this pull request
@Holoscopecheck Holoscopecheck requested review from Cyndaquil1999 and celorind and removed request for Cyndaquil1999 and celorind July 2, 2024 07:49
@henacken henacken changed the title dockerfileを書いていきます WIP dockerfileを書いていきます Jul 2, 2024
@henacken
Copy link
Collaborator

henacken commented Jul 2, 2024

開発途中でプルリクを出す場合、プルリクのタイトルの先頭に作業中の意味のWIP(Work In Progress)を書いとくといいですよ. 😉
(私は書くの忘れてましたが、、、)
誤マージを防げます。
追記: すいません、私が勝手にWIPつけてしまいました。作業が終わったら、タイトルのWIPを外すのお忘れなく👍

@Holoscopecheck
Copy link
Collaborator Author

@henacken WIPありがとうございます。

自分の手元では動いたのですが、これでマージしても大丈夫でしょうか。

@nona-takahara
Copy link
Contributor

マージはレビュー後にレビュアーが行います。
代わりに、Pull Requestタイトルの「WIP(Work In Progress)」を外して、作業が完了したとコメントしておいてください。

@Holoscopecheck Holoscopecheck changed the title WIP dockerfileを書いていきます dockerfileを書いていきます Jul 5, 2024
@Holoscopecheck
Copy link
Collaborator Author

@nona-takahara わかりました、ありがとうございます

作業完了しました。レビューお願いします

@Cyndaquil1999
Copy link
Contributor

正常に動いていますがコンテナ内とwsl環境でタイムゾーンのずれがあって、今後影響があるかもしれないので、後から修正タスクがあるかもしれないです。

@Cyndaquil1999 Cyndaquil1999 merged commit 7b972ee into main Jul 7, 2024
1 check passed
@Nlkomaru
Copy link
Contributor

Nlkomaru commented Jul 8, 2024

npm run devは最適化されなくて遅いので、npm run buildでビルドした後、npm run startなどで提供したほうが良いかも。

@Nlkomaru Nlkomaru deleted the feat/setup-docker branch September 16, 2024 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

フロント側でDockerで作業できるようにする
5 participants