Skip to content

chore: 빌드 인프라 및 마이그레이션 구조 재정비#33

Merged
Sean-mn merged 16 commits intodevelopfrom
refactor/claude
Apr 2, 2026
Merged

chore: 빌드 인프라 및 마이그레이션 구조 재정비#33
Sean-mn merged 16 commits intodevelopfrom
refactor/claude

Conversation

@Sean-mn
Copy link
Copy Markdown
Contributor

@Sean-mn Sean-mn commented Apr 2, 2026

📚작업 내용

  • Docker 빌드 구조를 deploy/ 폴더로 분리 (dev.dockerfile, prod.dockerfile, compose.dev.yaml, compose.prod.yaml)
  • CI/CD 워크플로우에서 prod.dockerfile 경로 반영
  • EF Core 초기 테이블 생성 마이그레이션 추가 (기존 마이그레이션 대체)
  • User 테이블 스키마를 game_user에서 auth로 변경 및 마이그레이션 추가
  • 개발 환경 DB 포트(5434) 및 Redis 포트(6381) 변경
  • dotnet SDK 버전 고정 (global.json, 9.0.201)
  • Gemini commit/PR 명령어 템플릿 추가
  • Claude 허용 명령어 목록 확장 (cmd.exe, dotnet --list-sdks, dotnet list:*)

◀️참고 사항

기존 PushAndPull/PushAndPull/Dockerfile 삭제 후 deploy/ 폴더로 빌드 파일 통합. User 테이블의 스키마명이 game_userauth로 변경되어 DB 마이그레이션 적용 필요.

✅체크리스트

[ ]안에 x를 작성하면 체크박스를 체크할 수 있습니다.

  • 현재 의도하고자 하는 기능이 정상적으로 작동하나요?
  • 변경한 기능이 다른 기능을 깨뜨리지 않나요?

추후 필요한 체크리스트는 업데이트 될 예정입니다.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request establishes a comprehensive containerized environment by adding Docker Compose configurations and specialized Dockerfiles for development and production. It also updates the .NET SDK version, configures AI-assisted prompts for git commits and PR generation, and refactors the database schema to use an 'auth' schema for user-related tables. Feedback focuses on correcting a networking issue in the development compose file where service names should be used instead of localhost for inter-container communication, and suggests consolidating migration steps to avoid redundant schema renames.

@Sean-mn Sean-mn merged commit 66e10a9 into develop Apr 2, 2026
4 checks passed
@Sean-mn Sean-mn deleted the refactor/claude branch April 2, 2026 09:24
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.

1 participant