Skip to content
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

fix: github Action 환경에서 FirebaseCrashlytics 이슈 수정해요 #735

Merged
merged 15 commits into from
Dec 29, 2024

Conversation

Do-hyun-Kim
Copy link
Collaborator

🔵PR을 올리기 전 아래 사항을 확인해주세요.

  • 구현한 로직과 기능이 올바르게 작동되는지 충분히 테스트해주세요.
  • 코드의 성능이나 메모리 효율성이 적절하게 고려되었는지, 불필요한 코드가 없는지 검토해주세요.
  • 이번 PR에서 구현된 주요 기능이나 해결된 문제에 대해 자세히 서술해주세요.
    (위 내용은 지워주세요)

😽 Firebase Crashlytics Resolve 했을 때 Github Action 환경에서 컴파일 에러 나는 이슈 수정해요

  • Github Action 환경에서 Firebase Crashlytics라이브러리를 Resolve 할 때 FIRCrashlytics.m 파일에 있는 내부 메서드를 가져오지 못하는 이슈를 해결
  • 확인해 보니깐 FirebaesRemoteConfig 일부 함수를 못 가져오는 현상인데 Yapp 프로젝트에서 동일한 이슈가 나타나서 Tuist Version 4.8.1로 업그레이드하여 해결했습니다.

Xcode Clean Build 시 Dependency Recycle 이슈 수정해요

  • Core 모듈이 전역적으로 많이 사용하다 보니 의존성 Recycle이 발생하는 현상이 나타났습니다.
  • Logging 같은 모듈은 Util 모듈을 추가해서 Core 모듈에 결합도를 낮추도록 했습니다.

🛠️Tuist Util 모듈 추가

  • Tuist에 Util 모듈 추가하여 이번에 추가된 Firebase Crashlytics, Firebase Analytics 관련 메서드를 Util 모듈 내부에 구성했습니다.

🛠️ Github Action Mac OS, Xcode 버전 업그레이드

  • 기존 Github Action 실행 버전이 Ventura였는데 Sequoia로 업그레이드했습니다.
  • Xcode Version도 동일하게 기존에는 15.0.1 환경에서 실행했으나 16.0.1 환경에서 실행하도록 수정했습니다.

✅ 테스트 케이스

  • Github Action 실행 시 배포가 되는지 확인해요
  • dsym파일이 업로드되는지 확인해요

🙏🏻아래와 같이 PR을 리뷰해주세요.

  • PR 내용이 부족하다면 보충 요청해주세요.
  • 코드 스타일이 팀의 규칙에 맞게 작성되었는지, 일관성을 유지하고 있는지 확인해주세요.
  • 코드에 대한 문서화나 주석이 필요한 부분에 적절하게 작성되어 있는지 확인해주세요.
  • 구현된 로직이 효율적이고 올바르게 작성되었는지, 아키텍처를 잘 준수하고 있는지 검토해주세요.
  • 네이밍, 포매팅, 주석 등 코드의 일관성이 유지되고 있는지 확인해주세요.

@Do-hyun-Kim Do-hyun-Kim added 🐞 BUG ⚒️ SETTING 개발 환경 세팅 (Tuist, CI/CD) 🦊 도현 labels Dec 27, 2024
@Do-hyun-Kim Do-hyun-Kim requested a review from akrudal December 27, 2024 11:18
@Do-hyun-Kim Do-hyun-Kim self-assigned this Dec 27, 2024
@Do-hyun-Kim Do-hyun-Kim merged commit da9b6d0 into develop Dec 29, 2024
@Do-hyun-Kim Do-hyun-Kim deleted the fix/#734-firebase-build-modify branch December 29, 2024 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BUG ⚒️ SETTING 개발 환경 세팅 (Tuist, CI/CD) 🦊 도현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant