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

🔀 :: (#470) 도메인 모듈 분리 작업 간 누락된 의존성 추가 #471

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

KangTaeHoon
Copy link
Contributor

@KangTaeHoon KangTaeHoon commented Mar 11, 2024

💡 배경 및 개요

import ooDomainInterface 선언이 에러가 나야 정상인데
그냥 넘어감. 클린빌드 돌려가며 누락된 의존성 부여

Resolves: #470

📃 작업내용

  • UserDomainInterface에 SongsDomainInterface 디펜던시 추가
  • PlayListDomainInterface에 BaseDomainInterface 디펜던시 추가

🙋‍♂️ 리뷰노트

✅ PR 체크리스트

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

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

🎸 기타

Copy link

github-actions bot commented Mar 11, 2024

✅ Successful finished SwiftLint

@github-actions github-actions bot added 🐞 Bug 버그 관련 ♻️ Refactor 코드 리팩토링 labels Mar 11, 2024
Copy link

✅ 이슈와 PR의 Labels 동기화를 성공했어요!

@KangTaeHoon KangTaeHoon merged commit a6af7e4 into develop Mar 11, 2024
5 checks passed
@KangTaeHoon KangTaeHoon deleted the fix/470-add-dependency branch March 11, 2024 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug 버그 관련 ♻️ Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

도메인 모듈 분리작업 간 누락된 의존성 부여
3 participants