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

[버그] withUnretained 걷어내기 #73

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

leeari95
Copy link
Owner

@leeari95 leeari95 commented Nov 13, 2024

📕 Issue

fix #60 #63

📙 작업 내역

구현 내용 및 작업 했던 내역

  • withUnretained를 사용하지 않는 방향으로 코드 개선
    • Be careful when using this operator in a sequence that has a buffer or replay, for example share(replay: 1), as the sharing buffer will also include the provided object, which could potentially cause a retain cycle.
  • 약한 참조를 언래핑이 불필요한 곳은 언래핑을 시도하지 않도록 수정

📘 작업 유형

  • 신규 기능 추가
  • 버그 수정
  • 리펙토링
  • 문서 업데이트

📋 체크리스트

  • Merge 하는 브랜치가 올바른가?
  • PR과 관련없는 변경사항이 없는가?
  • 내 코드에 대한 자기 검토가 되었는가?
  • 사용자 가이드 업데이트가 필요한가?
    • 사용자 가이드를 업데이트 하였는가?

@leeari95 leeari95 added the fix 버그 혹은 에러사항 개선 label Nov 13, 2024
@leeari95 leeari95 self-assigned this Nov 13, 2024
@leeari95 leeari95 changed the base branch from main to develop November 13, 2024 14:58
@leeari95 leeari95 changed the title 🔧 [chore] 기존 워크플로우 테스트 작업 제거 및 Tuist에 맞게 빌드하도록 수정 [버그] withUnretained 걷어내기 Nov 13, 2024
@leeari95 leeari95 merged commit 7184f1d into develop Nov 13, 2024
1 check passed
@leeari95 leeari95 deleted the fix/withUnretained branch November 13, 2024 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그 혹은 에러사항 개선
Projects
Status: ✅ Done
1 participant