Skip to content

Conversation

code-yeongyu
Copy link
Owner

배경

작업 내용

  • main 실행시 광고를 실행시킵니다.

리뷰노트

  • show_ad 안의 로직이 복잡합니다. 그러나 한번만 사용되고 있어서, 모듈화 되거나 추상화 되기엔 좋지 않다고 생각했습니다. (함수로 wrapping 했을때 퍼포먼스가 떨어지는 로직이기도 합니다.)
  • 불가피하게 구현에 대한 주석을 추가했습니다.

@code-yeongyu
Copy link
Owner Author

상세한 PR 과 설명까지 감사합니다!

@code-yeongyu code-yeongyu merged commit c8532c4 into master Feb 6, 2023
@code-yeongyu code-yeongyu deleted the feature/show-ad branch February 6, 2023 09:49
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