Skip to content

ci: 환경변수 인젝션 수정#216

Merged
kimsaram32 merged 1 commit intomainfrom
features/ci-fix-environment-variable
Dec 8, 2025
Merged

ci: 환경변수 인젝션 수정#216
kimsaram32 merged 1 commit intomainfrom
features/ci-fix-environment-variable

Conversation

@kimsaram32
Copy link
Copy Markdown
Collaborator

@kimsaram32 kimsaram32 commented Dec 8, 2025

개요

빌드 시 환경변수가 적용되지 않는 오류를 수정하였습니다.

Summary by CodeRabbit

릴리스 노트

  • Chores
    • Docker 빌드 프로세스의 명시적 빌드 컨텍스트 설정이 추가되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 8, 2025

개요

Docker 빌드 및 푸시 단계에 명시적 빌드 컨텍스트(context: .)가 추가되었습니다. 푸시 동작이나 태그 변경은 없으며 제어 흐름은 동일합니다.

변경사항

결합체 / 파일 변경 요약
GitHub Actions 워크플로우 배포 설정
\.github/workflows/deploy\.yml
Docker 빌드 및 푸시 단계의 with 섹션에 context: . 추가되어 빌드 컨텍스트를 명시적으로 지정

코드 리뷰 예상 소요시간

🎯 1 (매우 간단함) | ⏱️ ~2분

🐰 한 줄의 설정, 명확한 의도로,
빌드 컨텍스트 잘 정의하고,
워크플로우는 한층 단정하게,
작은 변화가 모여 안정성을 높인다! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive PR 제목이 변경사항과 부분적으로 관련이 있습니다. 제목은 '환경변수 인젝션 수정'을 언급하지만, 실제 변경사항은 Docker 빌드 단계에서 build context를 추가하는 것입니다. 제목을 'ci: Docker 빌드 context 명시화' 또는 'ci: 빌드 설정 개선'으로 변경하거나, PR 설명에서 빌드 context 추가가 환경변수 인젝션 문제를 어떻게 해결하는지 명확히 설명하십시오.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch features/ci-fix-environment-variable

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 597f7c9 and bae8be9.

📒 Files selected for processing (1)
  • .github/workflows/deploy.yml (1 hunks)
🔇 Additional comments (1)
.github/workflows/deploy.yml (1)

31-36: 環境 変数 주입이 제대로 적용되었는지 확인 필요합니다.

context: .를 추가하는 것은 빌드 컨텍스트를 명시하는 좋은 방법이지만, 단독으로는 환경 변수를 Docker 빌드 프로세스에 전달하지 못합니다. PR 목표가 "빌드 시 환경변수 적용" 이라면, VITE_REACT_APP_API_BASE_URL 변수가 실제로 Docker 빌드 단계에서 사용되고 있는지 확인이 필요합니다.

일반적으로 다음 중 하나의 방식이 필요합니다:

  1. docker/build-push-actionbuild-args 전달
  2. Dockerfile에서 ARG/ENV로 명시적으로 정의
  3. 기타 메커니즘을 통한 환경 변수 전달

Dockerfile 구성과 17-20번 줄에서 생성된 .env 파일이 빌드 프로세스에서 실제로 사용되는지 확인해 주세요.


Comment @coderabbitai help to get the list of available commands and usage tips.

@kimsaram32 kimsaram32 merged commit 15b62ac into main Dec 8, 2025
1 of 2 checks passed
@kimsaram32 kimsaram32 deleted the features/ci-fix-environment-variable branch December 8, 2025 12:58
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