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

[ISSUE-006] domain 모듈 추가 #22

Merged
merged 3 commits into from
May 19, 2022
Merged

Conversation

hoyahozz
Copy link
Member

@hoyahozz hoyahozz commented May 19, 2022

ISSUE


작업 내용

  • domain 모듈 생성
  • 생성된 app 모듈, presentation 모듈에 의존성 추가
  • 샘플 클래스 추가

실행 화면

Screen Shot Screen Shot

Check List

  • PR 제목은 [TICKET-{N}] PR 제목으로 작성
  • CI/CD 통과 여부
  • 1명 이상의 Approve 확인 후 Merge
  • 관련 이슈 연결

@hoyahozz hoyahozz added this to the 프로젝트 구축 milestone May 19, 2022
@hoyahozz hoyahozz self-assigned this May 19, 2022
@KwonDae
Copy link
Member

KwonDae commented May 19, 2022

LGTM 👍
적용한 kts 방식 어때여?
정호님이 예제로 하셨던거랑 조금 내용이 다를텐데 어떠셨나여

@hoyahozz
Copy link
Member Author

@KwonDae
전 괜찮은 것 같아요!

그런데 라이브러리 기준으로 리스트를 묶어놓다보니 만약 여러 모듈에서 리스트를 사용하게 될 경우 필요하지 않은 라이브러리까지 함께 사용하는 것이 강제되는? 그런 느낌이라서 오히려 모듈 단위로 리스트를 묶는 것도 괜찮겠다? 싶었어용
(물론 또 그러면 보일러 플레이트 코드가 추가되는 케이스가 생기는거라 뭐가 옳은지는 잘 모르겠네요..)

@KwonDae
Copy link
Member

KwonDae commented May 19, 2022

@hoyahozz

Compose, coroutine, glide, hilt 등 카테고리별로 묶여서 모듈 gradle엔 좀 깔끔한 느낌을 주고싶었는데
좀 더 상세하게 필요한 라이브러리만 넣기에는 한계가 있긴 하겠네용..

근데 라이브러리를 쓸 때 필요하지 않은 라이브러리가 공존하는 경우가 많으려나요?

@hoyahozz
Copy link
Member Author

@KwonDae

움 일단 이 부분은 회의 때 한번 더 얘기하면서 생각해보면 좋을 것 같아용
다만 확실히 되게 꼼꼼하게 짜셨구나... 는 느낄 수 있었습니당 👍 고생 많으셨어요!!

Copy link
Member

@KwonDae KwonDae left a comment

Choose a reason for hiding this comment

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

머지해주세용

@hoyahozz hoyahozz merged commit e7624f3 into develop May 19, 2022
@hoyahozz hoyahozz deleted the feature/issue-06-domain-module branch May 19, 2022 12:24
Copy link
Member

@KwonDae KwonDae left a comment

Choose a reason for hiding this comment

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

처음에 발견 못해서 미안해요 ㅎㅎ..

@@ -0,0 +1,4 @@
package co.kr.yapp.growth.domain.usecase
Copy link
Member

Choose a reason for hiding this comment

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

@hoyahozz
정호님 늦게봐서 미안해요
패키지명 com.yapp.growth 로 수정 부탁드립니당

}

java {
sourceCompatibility = JavaVersion.VERSION_1_7
Copy link
Member

Choose a reason for hiding this comment

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

여기도 자바 1.8 버전으로 수정 부탁해요

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.

[프로젝트 구축] domain 모듈 추가
2 participants