-
Notifications
You must be signed in to change notification settings - Fork 2
♻️ refactor: Dockerfile에 빌드 시점 환경변수 설정 추가 #112
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
Conversation
WalkthroughDockerfile의 빌더 단계에 Changes
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Dockerfile (2)
13-13: Dockerfile 주석 언어 컨벤션 검토
현재 주석이 한국어로 작성되어 있습니다. 프로젝트 전반에서 Dockerfile 주석이 영어로 통일되어 있는지 확인하고, 필요 시 영어 주석으로 변경을 고려하세요.
14-15: ARG 선언 시 기본값 부재로 인한 빌드 실패 위험
ARG NEXT_PUBLIC_API_URL과ARG NEXT_PUBLIC_TEAM_ID에 기본값이 없어, 빌드 시 빌더 인자를 지정하지 않으면 빈 문자열이 설정될 수 있습니다. 기본값 추가 또는 빌드 시점에 필수 인자가 주입되지 않으면 실패하도록 검증 스텝(RUN test -n "$NEXT_PUBLIC_API_URL" || exit 1)을 추가하는 방안을 제안드립니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Dockerfile(1 hunks)
🔇 Additional comments (1)
Dockerfile (1)
16-17: ENV로 ARG 주입 방식 검증
빌드 인자를 환경 변수로 재할당해 Next.js 앱에 주입하는 방식은 적절합니다. 다만 CI/CD 파이프라인 또는docker build커맨드에서--build-arg NEXT_PUBLIC_API_URL=... --build-arg NEXT_PUBLIC_TEAM_ID=...가 올바르게 설정되어 있는지 확인해 주세요.
📌 변경 사항 개요
Dockerfile의 빌드 시점 환경변수 설정을 개선하여 외부에서 동적으로 환경변수를 전달받을 수 있도록 수정.
✨ 요약
📝 상세 내용
빌드 시점에 환경변수 설정 (외부에서 전달받음, 기본값 없음)
ARG NEXT_PUBLIC_API_URL
ARG NEXT_PUBLIC_TEAM_ID
🔗 관련 이슈
🖼️ 스크린샷
✅ 체크리스트
💡 참고 사항
Summary by CodeRabbit