Closed rkdgusrnrlrl closed 6 years ago
코드리뷰 문제는 저도 겪고 있는데요 매일 매일 PR을 처리하지 않으면 PR 크기가 너무 커지더라구요
서로 의식적으로 처리해야겠다라는 생각이 중요한거 같아요
어떻게 해야할까요
@ChangJoo-Park PR 하나의 크기가 큰다는 건가요 PR 자체가 많다는 건가요? 우선 저의 경우에는 PR 크기가 큰게 문제 인것 같습니다.
한 PR에 코드 수정 된 부분이 너무 많으면 직관적으로 파약 하기 어려운 것 같습니다. 그렇다보니 아 잘 돼겠지 뭐 하는 경우가 생기는 것 같아요. 의식적으로 하더래도 우선 PR 하는 사람의 부담감을 줄여 주는 게 중요한것 같습니다.
그러먼 아무래도 커밋 메세지에 신경을 써야 되고, 풀 리퀘스트도 좀 잘 정리 해두면 도움이 되는 것 같습니다. 그래서 5분 10분 내로 심심할떄(?) 할 수 있는 소일거리 정도 느낌을 줄 수 있으면 좋을 것 같아요(바람)
하지만 중요한거는 리뷰 자체도 하나의 중요한 일감(코딩과 같이)라고 마인드가 중요 하고, 뭔가 리뷰 자체를 즐길 수 있는 환경이 된다면 더 좋을 것 같습니다. 그런데 어떻게 하면 좀더 그런 분위기를 만들수 있을지 아직 저도 잘 모르겠습니다. ㅎㅎ
여기서는 pr하나의 크기였어요 한 pr이 계속 개발되어야하는 부분의 일부였을때 그 브랜치에서 계속 개발하다보면 최초 브랜치 의도와 다르게 되는 경우가 발생해서요
아직 프로젝트가 어느정도 수준까지 올라온 상태가 아니라 발생하는 문제일거라 생각하고 있긴해요
그래서 시니어개발자가 베이스를 다 깔아주어서 다른사람들이 독립적으로 개발할 수 있도록 해야하지 않을까 생각해봤어요 여기에는 테스트, 코드 보일러플레이트 등이 있을거같아요
깃헙을 쓰시면 #이슈번호 를 커밋메시지에 강제해보는것도 괜찮지 않을까요
http://ohyecloudy.com/pnotes/archives/git-add-branch-name-to-commit-message/
https://robots.thoughtbot.com/better-commit-messages-with-a-gitmessage-template
@ChangJoo-Park 코드 보일러 플레이트는 정말 큰 도움이 될 것 같습니다. 코드의 일관성만 보장되도 한결 보기 수월 꺼 같아요 ㅎㅎ 브랜치 자체가 이슈 기반으로 가고 있는데, 딱히 커밋 메세지에 이슈 번호를 강제 하진 않는데, 혹시 커밋 메세지에 이슈 번호를 넣으면 더 도움이 될까요??
나중에 머지되었을때 작업한 브랜치는 지우게 되는데 그러면 어떤 이슈에서 작업한 내용인지 찾기 어려울거같다는 생각이었어요
padding
이 언제 완료가 되는 지 모르겠음
목적
상세 내용