Skip to content

Conversation

@maehwasoo
Copy link
Member

@maehwasoo maehwasoo commented Jan 9, 2026

📌 Summary

GitHub Actions 배포 워크플로를 main/develop 모두에서 동작하도록 정리하고
배포 대상 레포/소유자 설정을 GitHub Repo Variables로 전환했습니다.

또한 카카오 로그인 OAuth 흐름에서 env=local|preview|prod를 지원하도록 로그인/콜백 및 API 타입을 정리했습니다.

Firebase Analytics는 VITE_ENABLE_FIREBASE_ANALYTICS 플래그로 명시적으로 켜고 끌 수 있게 했습니다.

📄 Tasks

CI/CD

  • .github/workflows/deploy.yml 트리거를 main/develop로 확장하고 target-branch를 ${{ github.ref_name }}로 변경
  • 배포 대상(destination owner/repo)을 ${{ vars.DEPLOY_DESTINATION_OWNER }} / ${{ vars.DEPLOY_DESTINATION_REPO }}로 전환하여 하드코딩 제거

Auth

  • 호스트명 기반 local|preview|prod 판별 유틸(getAuthEnvironment) 및 타입(AuthEnvironment) 추가

Kakao OAuth

  • 로그인/콜백에서 env=local|preview|prod로 통일하고getKakaoLogin/useKakaoLoginMutation도 AuthEnvironment 타입으로 정리

Analytics & ETC

  • Firebase Analytics 초기화를 VITE_ENABLE_FIREBASE_ANALYTICS === 'true'일 때만 수행하도록 변경하고 vite-env.d.ts 타입 추가
  • upstream 자동 동기화 워크플로(sync-upstream.yml) 제거

🔍 To Reviewer

📸 Screenshot

@coderabbitai
Copy link

coderabbitai bot commented Jan 9, 2026

📝 Walkthrough

Summary by CodeRabbit

변경 사항

  • Chores
    • 업스트림 동기화 GitHub Actions 워크플로우가 제거되었습니다. 이전에는 메인 및 개발 브랜치를 정기적으로 동기화하는 자동화된 프로세스를 관리했습니다.

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

개요

GitHub Actions 워크플로우 파일 .github/workflows/sync-upstream.yml이 삭제되었습니다. 이 워크플로우는 15분마다 또는 수동으로 트리거될 때 GitHub API를 통해 업스트림 브랜치(main, develop)를 동기화하는 작업을 수행했습니다.

변경 사항

응집군 / 파일(들) 변경 요약
워크플로우 삭제
\.github/workflows/sync-upstream\.yml
전체 파일 삭제 (64줄 제거). 주기적 업스트림 동기화 워크플로우 및 병합 로직 완전 제거

예상 코드 리뷰 난이도

🎯 1 (Trivial) | ⏱️ ~3 분

추천 레이블

🌐 Deploy

추천 검토자

  • earl9rey
  • soyyyyy
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목 '[deploy] Vercel 배포 마이그레이션'은 요구사항을 만족하며, 변경사항의 핵심(Vercel 배포 마이그레이션)을 명확히 반영하고 있습니다.
Description check ✅ Passed PR 설명은 변경사항과 관련이 있으며, 주요 작업 항목(CI/CD, Auth, Kakao OAuth, Analytics)과 sync-upstream.yml 제거를 상세히 기술하고 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ee93ffa and 15ed0fb.

📒 Files selected for processing (1)
  • .github/workflows/sync-upstream.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/sync-upstream.yml

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.

@github-actions
Copy link

github-actions bot commented Jan 9, 2026

빌드 결과

빌드 성공 🎊

@maehwasoo maehwasoo marked this pull request as ready for review January 9, 2026 16:32
@maehwasoo maehwasoo requested a review from a team as a code owner January 9, 2026 16:32
@maehwasoo maehwasoo self-assigned this Jan 9, 2026
@maehwasoo maehwasoo merged commit dbe44f2 into develop Jan 9, 2026
3 checks passed
@maehwasoo maehwasoo added 🤙 경호 웹 36기 엄경호 🌐 Deploy 배포 관련 labels Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🌐 Deploy 배포 관련 🤙 경호 웹 36기 엄경호

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deploy] Vercel 배포 마이그레이션

2 participants