PR Template 개선에 대한 아이디어 #507
HanGyeongjun
started this conversation in
General
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
최근 PR 작성 시 현재 PR 템플릿에 있는 항목으로만은 다음과 같은 문제가 있다고 느꼈습니다.
내용이 짧거나 단순한 작업이면, 현재
관련 이슈, 구현/변경 사항, 스크린샷
3가지 항목으로도 이해가 가능합니다.하지만 기능이 복잡해지거나 길어지면 추가 컨텍스트 없이는 빠른 시간 안에 코드를 이해하는 것이 상대적으로 어려워지는 문제가 생길 수 있습니다.
또한 Assignee가 의도하지 않은 방향으로 이해하거나 필요한 테스트를 하지 못하는 상황이 발생할 수 있다고 생각합니다.
현재 템플릿을 유지하더라도 Assignee는 원한다면 다른 내용이 포함된 구체적인 PR을 작성할 수 있습니다.
다만 이 방법은 Assignee가 더 많은 시간을 할애해야 하고, 각 PR간 일관성이 떨어지게 된다는 또 다른 문제를 야기합니다.
그래서 현재 탬플릿보다 보다 내용을 구체적으로 작성할 수 있는 개선된 템플릿이 필요하다고 생각합니다.
이를 위해 몇 가지 레퍼런스를 조사했습니다.
뱅크샐러드 코드리뷰 문화
공비서 코드리뷰 문화
이 중 두 번째 공비서의 PR 템플릿에서 참고할 만한 부분이 많아보입니다.
우리 프로젝트에 봇, 자동화 태그 등을 적용하기에는 현실적인 어려움이 있기에 PR 템플릿 정도만 참고하면 PR 내용의 개선, 코드리뷰의 속도와 정확성 측면에서 긍정적인 영향을 줄 것 같습니다.
그래서 다음과 같이 변경된 PR 템플릿을 제안합니다.
기존 템플릿보다 구체적 작성이 가능하되, 작성에 지나치게 오랜 시간이 걸리지 않도록 조정했습니다.
더 좋은 템플릿 구성이나, 추가 혹은 제거할 항목에 대한 아이디어가 있으면 코멘트로 남겨 주시면 감사하겠습니다!
Beta Was this translation helpful? Give feedback.
All reactions