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

feat(auth): add #8

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat(auth): add #8

wants to merge 4 commits into from

Conversation

n4rvs3
Copy link
Member

@n4rvs3 n4rvs3 commented Jun 1, 2023

No description provided.

@n4rvs3 n4rvs3 requested a review from tingtt June 1, 2023 06:57
@n4rvs3 n4rvs3 mentioned this pull request Jun 1, 2023
Copy link
Member

@tingtt tingtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

recoil, @types/recoil は devDependencies でお願いします

next 以外は devDependencies で良いかと
(CI/CDパイプラインやDocker上での実行に勝手が良いので)

@tingtt tingtt closed this Jun 1, 2023
@tingtt
Copy link
Member

tingtt commented Jun 1, 2023

まちがえた

@tingtt tingtt reopened this Jun 1, 2023
@tingtt tingtt changed the title feat: add-auth feat(auth): add Jun 1, 2023
@n4rvs3
Copy link
Member Author

n4rvs3 commented Jun 5, 2023

recoil, @types/recoil は devDependencies でお願いします
next 以外は devDependencies で良いかと
(CI/CDパイプラインやDocker上での実行に勝手が良いので)

これについて、理屈は理解してるんだけど実際のメリットってどんな感じなのか分からないからざっくりで良いから教えてもらえるとたすかる

特段気にしてはないんだけどRecoil公式であったりその他解説記事諸々等読んでみてもdevDependenciesじゃなくてDependenciesとしてインストールしてたから気になって👀(多分動きはする)

@n4rvs3
Copy link
Member Author

n4rvs3 commented Jun 5, 2023

"dependencies": Packages required by your application in production.
"devDependencies": Packages that are only needed for local development and testing.

日本語訳で

"dependencies": 本番環境のアプリケーションに必要なパッケージ。
"devDependencies": ローカルの開発とテストにのみ必要なパッケージ。

シンプルにdevDependenciesとDependenciesの使い分けを意識したことがなかったから調べてみたけど、上記がイマイチしっくりこない、この理論で行くとdependenciesが当てはまりそうな気がして

@tingtt
Copy link
Member

tingtt commented Jun 5, 2023

※ 確かに普通ではやってない気がする。

next build で生成されたファイルを next start で実行する際に使用されるのは基本的に next のみなので dependencies には next のみの記述にしています。
output: "standalone" でビルドする場合はどのみち node server.js するだけなので関係ないですが。

↓ 以下説明

Dockerでマルチステージビルドする場合の例

  1. build ステージ

    • pnpm install & pnpm build
  2. runner ステージ

    • buildステージから .next/node_modules/ をコピー
    • npm start
      ※ 全てのパッケージを内包

に比べて、

  1. build ステージ

    • pnpm install & pnpm build
  2. deps ステージ

    • pnpm install --prod
  3. runner ステージ

    • buildステージから .next/ をコピー
    • deps ステージから node_modules/ をコピー
    • npm start
      next のみ内包

という感じでコンテナイメージの容量削減ができる。
(コンテナイメージ限らず デプロイ先で実行する場合の容量削減をpackge.jsonだけで明記できる

Copy link
Member

@tingtt tingtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approveしておきます。

@n4rvs3
Copy link
Member Author

n4rvs3 commented Jun 18, 2023

取り急ぎnext-auth / github でログイン(jwt管理)追加した、ディレクトリ構造等まだ決まってないからそれっぽくファイル作ってべた書きしている(量少ないし)

構造とか命名規則ちゃんとしたら細分化ヨシ

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.

2 participants