Merged
Conversation
YoungJJun
approved these changes
Apr 5, 2026
Collaborator
YoungJJun
left a comment
There was a problem hiding this comment.
3주차 리뷰
-
DI 방법 중 생성자 주입이 권장되는 이유, 순환참조 감지 시점까지 조사해주신 점 너무 좋아요.
-
표까지 만들어서 설명해주셔서 표 하나로 쉽게 이해할 수 있는것 같아요.
-
Lombok 이 무엇이고 왜 어노테이션으로 대체 가능한지 찾아보시면 좋을 것 같아요.
코드에서@Getter,@Setter등 붙이는거 보셨을 것 같은데 어떻게 동작하는건지 간단히 알아보시면 됩니다! -
미션 영역별로 나눠서 정리해주신 점이 좋았습니다.
-
워크북 내용과 별개로 깃 사용과 관련해서 설명드리자면..
이번 키워드 커밋에서 이전 주차의 폴더명을 수정하신 것 같습니다.
폴더명을 수정 후 바로 키워드를 진행하시고 커밋을 해주셨습니다. 그러면 keyword 커밋에 폴더명을 변경한 내용이 들어가겠죠? 물론 단순히 파일을 작성하고 커밋하는 워크북 상황에서는 큰 문제가 없지만 만약에 코드를 작성할 때 폴더명이나 파일 이름등을 수정하는 상황이라면- chore (or docs): 폴더명 오타 수정
- feat: ~개발
이런식으로 두 개의 커밋으로 나눠서 진행해주시면 좋습니다!
제이 3주차 수고하셨습니다~
다음주도 화이팅입니다! 👏
Author
|
3주차 피드백 감사합니다. |
This file contains hidden or 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
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.
✅ 워크북 체크리스트
✅ 컨벤션 체크리스트
📌 주안점