Closed chws0508 closed 6 months ago
tmdgh1592
commented
6 months ago 고생하셨습니다! 전체적으로 패키지 구조가 명확해졌다는 느낌이 들어요. 이전에는 data 패키지 하위에 너무 많은 디렉토리가 있었는데, 지금은 경우에 따라 잘 묶이고 분리된 것 같아요!
+) 코멘트를 드리자면 프로젝트에 Module이 그렇게 많지 않은데 di 관련된 파일들이 너무 분산되어 있다고 느꼈어요. 특정 모듈을 수정하기 위해 여러번 클릭해서 들어가는게 번거롭다고 느껴졌습니다..! di 패키지는 하나로 모아 놓는 것은 어떨까요??
chws0508
commented
6 months ago 고생하셨습니다! 전체적으로 패키지 구조가 명확해졌다는 느낌이 들어요. 이전에는 data 패키지 하위에 너무 많은 디렉토리가 있었는데, 지금은 경우에 따라 잘 묶이고 분리된 것 같아요!
+) 코멘트를 드리자면 프로젝트에 Module이 그렇게 많지 않은데 di 관련된 파일들이 너무 분산되어 있다고 느꼈어요. 특정 모듈을 수정하기 위해 여러번 클릭해서 들어가는게 번거롭다고 느껴졌습니다..! di 패키지는 하나로 모아 놓는 것은 어떨까요??
저도 토마스 의견처럼 Hilt의 모듈이 모듈에서 제공하는 것과 가까이 있는 게 추후 유지보수와 관심사 분리 측면에서 좋아 보입니다
️⃣ 연관된 이슈
📝 작업 내용
Data Package Refactoring
스크린샷 (선택)
예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)
예상 소요 시간 : 2시간 실제 소요 시간 : 2시간
💬 리뷰어 요구사항 (선택)