Skip to content

refactor: simplify environment variable management#380

Merged
manNomi merged 1 commit intomainfrom
fix/vercel-build
Jan 25, 2026
Merged

refactor: simplify environment variable management#380
manNomi merged 1 commit intomainfrom
fix/vercel-build

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Jan 25, 2026

  • Remove .env.development and .env.preview files
  • Use .env for stage API (default for PR/Preview/main)
  • Keep .env.production for production API reference
  • Configure release.yml to inject production env vars with --build-env
  • Add .env.guide.md for environment setup documentation

관련 이슈

  • resolves: #이슈 번호

작업 내용

특이 사항

리뷰 요구사항 (선택)

- Remove .env.development and .env.preview files
- Use .env for stage API (default for PR/Preview/main)
- Keep .env.production for production API reference
- Configure release.yml to inject production env vars with --build-env
- Add .env.guide.md for environment setup documentation
@vercel
Copy link

vercel bot commented Jan 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
solid-connection-web Ready Ready Preview, Comment Jan 25, 2026 10:57am

@coderabbitai
Copy link

coderabbitai bot commented Jan 25, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

이 변경사항은 Next.js 프로젝트의 환경 설정 및 배포 자동화를 정리하는 작업입니다. 주요 변경 내용은 다음과 같습니다:

  1. .env 파일 업데이트: Stage API 관련 환경 변수(NEXT_PUBLIC_WEB_URL, NEXT_PUBLIC_API_SERVER_URL, NEXT_PUBLIC_KAKAO_JS_KEY)를 추가하고 설정 설명을 담은 주석 작성.
  2. .env.development 파일 정리: 개발 환경 전용 설정 변수들을 제거하여 로컬 개발 설정을 .env.local로 통합.
  3. .env.guide.md 문서 신규 작성: 세 가지 환경 파일(.env, .env.production, .env.local)의 역할과 배포 흐름, 모범 사례를 상세히 문서화.
  4. .github/workflows/build.yml 추가: 메인 브랜치 푸시 시 자동으로 Vercel 프로덕션 배포를 수행하는 워크플로우 신규 구성.
  5. .github/workflows/release.yml 개선: 빌드 시점에 프로덕션 환경 변수를 명시적으로 주입하도록 release 워크플로우 수정.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@manNomi manNomi merged commit 24ef5c2 into main Jan 25, 2026
4 of 5 checks passed
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