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.
✒️ 관련 이슈번호
Key Changes 🔑
저희는 패키지 컨벤션으로 Layerd Architecture 구조로 패키징을 했습니다.
하지만 개발 초 서로 멋 모르고 파일을 생성했고, 그 결과 명확하지 않은 패키지 구조를 가지고 있단 생각이 들었습니다.
이를 해결하기 위해 presentation, service, persistence 패키지를 만들어서 계층형 패키지임을 명시했습니다.
또한, 각 계층에서 사용하는 파일들을 구분해서 각 계층 별 config와 common 패키지를 만들었습니다.
마지막으로 모든 계층에서 공통적으로 사용하는 파일들은 common 패키지로 이동했고, 외부 라이브러리를 사용하는 파일들은 infra 패키지로 이동했습니다.
To Reviewers 📢
개선하고 싶은 점
현재 이미지와 같이 service 계층에서 presentation 계층을 참조하고 있는 것을 볼 수 있고, 이로 인해 service 계층과 presentation 계층이 양방향으로 참조되고 있다고 볼 수 있습니다. 따라서 service 계층이 presentation 계층을 참조하지 않는 작업을 제안하고 싶습니다..!
Read Me에 들어 갈 패키지 구조
이렇게 작성하려 합니다.
추가하거나 보완하고 싶은 부분 있으면 리뷰 남겨주세요!!