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

[레거시 코드 리팩터링 - 4단계] 후니(최재훈) 미션 제출합니다. #452

Open
wants to merge 2 commits into
base: jayjaehunchoi
Choose a base branch
from

Conversation

jayjaehunchoi
Copy link

안녕하세요 어썸오

멀티 모듈 적용해보았습니다.

  1. api와 관련한 로직은 api 모듈에 배치하였고 여기에만 executable Jar 파일이 생성되도록 하였습니다.
  2. core에는 domain, repository 관련 클래스를 배치하였고, repository도 다른 구현체가 생긴다면 다른 모듈로 분리하게 될 것 같습니다. 우선은 일반적으로 사용하는 jpaRepository 같은 경우에는 함께 core에 두는 게 좋겠다고 생각했어요.
  3. common에는 exception 내용들만 추가해두었습니다.
  4. https://toss.tech/article/how-to-manage-test-dependency-in-gradle 링크를 참고해서 testFixture에 대한 의존성관리도 해보았습니당.

모든 모듈 분리 기준은 현재 애플리케이션 상황을 기준으로 잡았습니다!
리뷰 잘 부탁드려용

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.

1 participant