Skip to content

[mainにmergeしない]42Repo#326

Closed
soragogo wants to merge 9 commits intomainfrom
feat/for-submit
Closed

[mainにmergeしない]42Repo#326
soragogo wants to merge 9 commits intomainfrom
feat/for-submit

Conversation

@soragogo
Copy link
Copy Markdown
Collaborator

@soragogo soragogo commented Jul 13, 2025

このプルリクエストはmainにmergeしません。
42のリポジトリに提出するためのブランチを見える化するためのものです。

リポジトリに.envファイルがないので(これは正しい状態)、GitHubActionのmakeは必ず失敗します。
やることリスト👉 #259

Copilot AI review requested due to automatic review settings July 13, 2025 03:37
@soragogo soragogo marked this pull request as draft July 13, 2025 03:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

提出用にポート設定を調整するPRだよ〜✨

  • nginxのSSLリスニングポートを443→8002に変更🥳
  • docker-composeのPostgres/Redisポートをコメントアウトし、nginxのポートマッピングを8002に更新🔒
  • フロントエンド環境変数(.env.sample)のAPI/WS URLに:8002を追加📡

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
reverseproxy/nginx.conf SSLポートを8002に変更
docker-compose.yml Postgres/Redisのポート定義をコメントアウト/nginxを8002に変更
.env.sample VITE_API_URL、VITE_WEBSOCKET_URLにポート8002を追記
Comments suppressed due to low confidence (3)

docker-compose.yml:23

  • 不要になったTODOコメントとコメントアウトされたポート定義は削除して、設定ファイルをクリーンに保つことを検討してください。
    # ports:

docker-compose.yml:44

  • [nitpick] READMEやドキュメントに、リバースプロキシがホスト側の8002ポートで稼働する旨を追記すると利用者に分かりやすくなります📖
      # - "80:80"

.env.sample:10

  • [nitpick] .env.sampleにポート番号を指定したURLを更新していますが、用途やカスタマイズ方法をコメントで補足すると導入時の混乱を防げます💡
VITE_API_URL=https://localhost:8002

@jizots jizots linked an issue Jul 13, 2025 that may be closed by this pull request
5 tasks
@jizots jizots changed the title ⚠️提出時用のPort設定など⚠️ [mainにmergeしない]42Repo Jul 20, 2025
@jizots jizots closed this Aug 6, 2025
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.

42のPCでプロジェクトを動かせるようにする

5 participants