f-lab-edu / used-market-server

[카카오, 넥슨 합격 / 이직 연봉 약 70% 상승] 번개장터 같은 중고거래 사이트
41 stars 3 forks source link

DB연동, Swagger lib 추가, Response, Message Service 구현, spring mvc 구조 설계 #7

Closed junshock5 closed 4 years ago

junshock5 commented 4 years ago

6

f-lab-dev commented 4 years ago
f-lab-dev commented 4 years ago

그리고 리뷰반영한 커밋메세지를 보면 Intellij에서 자동으로 생성해주는 커밋 메세지로 보이는데요. 커밋 메세지나 PR제목 같은 디테일한 곳에도 꼼꼼하게 신경써주시면 좋겠습니다.

이 프로젝트는 이력서에 들어있는 프로젝트 중에 소스코드와 기타 프로젝트관리 등을 조회할 수 있는 몇 안되는 프로젝트입니다.

커밋 메세지에 이슈번호를 달아준다거나 어떻게 메세지를 작성해야 이 커밋에 담긴 작업내용을잘 나타낼 수 있는지 계속 고민해보시면 좋을 것 같습니다.

평소에 이런 것을 잘 관리 안하시던 스타일이시면 스타일을 바꾸셔야할 것 같고요~

일반적으로 많은 분들이 기술을 이리저리 사용해보는건 화려해보이니까 그런 곳에 신경을 많이 쓰시는데 개발자는 이런 디테일과 꼼꼼함 또한 중요합니다. 그리고 항상 더 나은 방식이 있는지 고민하는 습관도 중요합니다.

물론 계속 프로젝트를 진행하면서 발전하는 모습을 보여주면 되니까 이 PR 하나만 놓고 얘기하기는 힘들고요. 다음 커밋, PR에서 디테일을 계속 신경써주시면 될 것 같아요~