[레거시 코드 리팩터링 - 4단계] 제이미(임정수) 미션 제출합니다 #805
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
carsago
approved these changes
Oct 29, 2023
Comment on lines
+1
to
+2
| bootJar { enabled = false } | ||
| jar { enabled = true } |
Comment on lines
1
to
+11
| rootProject.name = 'kitchenpos' | ||
|
|
||
| include 'app' | ||
| include 'core' | ||
| include 'domain-menu' | ||
| include 'domain-menugroup' | ||
| include 'domain-order' | ||
| include 'domain-ordertable' | ||
| include 'domain-ordertable' | ||
| include 'domain-product' | ||
| include 'domain-tablegroup' |
There was a problem hiding this comment.
고생하셨습니다.
사실 모듈을 나눠야할 필요가 있어서 나눴다기 보다는 미션 요구사항으로 나누라고해서 나눈거다 보니 어떻게 나누는 것보단 한 번 해본게 중요한 경험이라고 생각합니다 !!
그리고 이 과정에서 고민한 것들이 이후에 쓸모가 있지않을까용
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
안녕하세요 오리!
이제 정말 마지막 미션의 마지막 단계네요!
2단계의 추가적인 피드백도 수정 맟 답변달아 두었습니다.
이번 미션에서는 각 컨텍스트 별로 모듈을 분리해 진행해 보았습니다.
그런데 생각보다 모듈 설정이 어려워 분리에 계속 구조를 바꾸는 바람에
적절한 위치에 대해선 집중을 좀 못 하게 되어 잘한 것인지 잘 모르겠네요..!
특히, 테스트의 위치와 ui 패키지의 경우 app 모듈로 이동해야 하는 가에 대한 고민을 하고 있습니다.
테스트의 경우 원래 각 도메인 모듈에 넣으려고 했는데 이는 fixture를 어떻게 해야 할지 몰라 실패했습니다.
좀 우당탕탕 진행하긴 했지만 이번 리뷰도 잘 부탁드립니다!