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

🔀 :: (#419) MFA 도입을 위한 sugar 코드 도입 #422

Merged
merged 19 commits into from
Feb 25, 2024

Conversation

yongbeomkwak
Copy link
Member

💡 배경 및 개요

PR을 하게 된 문제상황, 배경 등 개요에 대해서 작성해주세요!

퍼블리싱의 경우 스크린샷/동영상도 추가해주면 좋아요!

Resolves: #419

📃 작업내용

PR에서 한 작업을 자세히 작성해주세요!

  • 이전에 사용한 Target 생성자들이 deprecated되지 않은 생성자로 교체
    • platform, deploymentTarget 변수를 이용한 Target은 현재 deprecated
    • destinations, deploymentTargets로 변경됨



스크린샷 2024-02-20 오후 11 44 03



  • makeModule 함수 대신 인터페이스, 데모앱등 다양한 모듈 의존관계를 조금 더 쉽게 구현하기 위한 Sugar 코드를 도입
    참고 자료(백튼님 템플릿)

🙋‍♂️ 리뷰노트

구현 시에 고민이었던 점들 혹은 특정 부분에 대한 의도가 있었다면 PR 리뷰의 이해를 돕기 위해 서술해주세요!

또한 리뷰어에게 특정 부분에 대한 집중 혹은 코멘트 혹은 질문을 요청하는 경우에 작성하면 좋아요!

e.g. 작업을 끝내야할 시간이 얼마 없어 확장성보다는 동작을 위주로 만들었어요! 감안하고 리뷰해주세요!

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@yongbeomkwak yongbeomkwak self-assigned this Feb 21, 2024
@yongbeomkwak yongbeomkwak added ♻️ Refactor 코드 리팩토링 ⚙ Setting 개발 환경 세팅 labels Feb 21, 2024
Copy link

github-actions bot commented Feb 21, 2024

✅ Successful finished SwiftLint

Copy link
Member

@baekteun baekteun left a comment

Choose a reason for hiding this comment

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

주석들같은경우에는 지우는거 어떤가요?
살려놓는게 필요할거같으면 살린 이유도 적어놓는게 좋을거같아용

@yongbeomkwak
Copy link
Member Author

주석들같은경우에는 지우는거 어떤가요? 살려놓는게 필요할거같으면 살린 이유도 적어놓는게 좋을거같아용

혹시 어디 쪽 주석 얘기하시는걸까요 ??

@baekteun
Copy link
Member

주석들같은경우에는 지우는거 어떤가요? 살려놓는게 필요할거같으면 살린 이유도 적어놓는게 좋을거같아용

혹시 어디 쪽 주석 얘기하시는걸까요 ??

image image

여기 둘이용

@yongbeomkwak
Copy link
Member Author

주석들같은경우에는 지우는거 어떤가요? 살려놓는게 필요할거같으면 살린 이유도 적어놓는게 좋을거같아용

혹시 어디 쪽 주석 얘기하시는걸까요 ??

image image
여기 둘이용

아 여기는 주말에 설명드리고 필요 없을 때 지워도 될까요 ?? 혹시 몰라서 주석처리 해놓은거라

@baekteun
Copy link
Member

아 여기는 주말에 설명드리고 필요 없을 때 지워도 될까요 ?? 혹시 몰라서 주석처리 해놓은거라

Copy link
Member

@kimdaehee0824 kimdaehee0824 left a comment

Choose a reason for hiding this comment

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

따따봉

@yongbeomkwak yongbeomkwak merged commit 1293d73 into 416-re-distribution-plugin Feb 25, 2024
3 checks passed
@yongbeomkwak yongbeomkwak deleted the feature/refactor_for_layer branch February 26, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링 ⚙ Setting 개발 환경 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants