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

feat: BibbiNetworkMonitor Legacy 코드 제거 #681

Merged
merged 6 commits into from
Oct 17, 2024

Conversation

Do-hyun-Kim
Copy link
Collaborator

@Do-hyun-Kim Do-hyun-Kim commented Oct 16, 2024

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

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

😽개요

  • 기존 BibbiNetworkMonitor Legacy코드 제거 및 BBNetworkEventMonitor로 EventMonitor 적용
  • BibbiNetworkMonitor 내부 BBLogger 코드 적용
  • BBLogger 모듈 내부 LogLevel, Log Method, Log ObservableType` 추가
  • FastLane app_store_build_number 메서드 추가
  • Tuist Marketing Version 1.2.3으로 수정 및 UILaunchStoryboardName 확장자 제거

🛠️작업 내용

BBLogger Module 구현

  • Logger Struct를 활용해서 BBLogger를 구현했습니다. Logger 같은 경우 MetaDataType에 따라 출력된 데이터를 확인할 수 있으며 Category는 해당 메서드 파라미터에 지정하도록 구현했습니다.
  • CategoryNested type으로 빼두지 않은 이유는 유동적이어야 하기 때문입니다. Logger의 장점은 필터링을 통해 원하는 출력값을 빠르게 조회할 수 있다는 점입니다. 하지만 Category를 임의로 지정을 해두면 한정적인 자원에서 필터링을 할 수밖에 없기 때문에 조회 가정에서 많은 어려움이 있습니다. 추가로 여러 case가 추가될 수도 있기 때문입니다.
  • FastLane에서 app_store_build_number 추가했습니다. 추가한 이유는 중간에 Xcode로 테플에 업로드 시 github Action으로 가져오는 BuildNumber와 매칭이 안되기 때문에 이미 테플에 올라가있는 BuildNumber를 활용해 증가시키도록 수정하였습니다.

BBLogger Module 출력 내용

  • 해당 Logger는 Logger를 호출한 Method Name, FileName, timestamp을 출력하도록 구현했습니다.
이미지①

|

✅테스트 케이스

  • LogMethod가 정확히 호출되는지 확인해요
  • BibbiNetworkMonitor에 로그가 출력되는지 확인해요

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

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

- BBNetworkEventMonitor BBLogger 출력구문 추가
- BBLogger 내부 Logger Method 추가
- BBLogger log ObservableType 추가
@Do-hyun-Kim Do-hyun-Kim self-assigned this Oct 16, 2024
@Do-hyun-Kim Do-hyun-Kim marked this pull request as draft October 16, 2024 12:00
@Do-hyun-Kim Do-hyun-Kim marked this pull request as ready for review October 16, 2024 15:16
@Do-hyun-Kim Do-hyun-Kim added the ⚒️ SETTING 개발 환경 세팅 (Tuist, CI/CD) label Oct 17, 2024
Copy link
Collaborator

@rlarjsdn3 rlarjsdn3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

깔끔하네요 수고하셨슴돠

@Do-hyun-Kim Do-hyun-Kim merged commit 51b167f into develop Oct 17, 2024
2 checks passed
@Do-hyun-Kim Do-hyun-Kim deleted the feat/#680-add-oslog-module branch October 17, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ FEATURE 기능 개발 ⚒️ SETTING 개발 환경 세팅 (Tuist, CI/CD) 🦊 도현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants