풀 리퀘스트와 코드 리뷰의 연극

Goatmire Elixir Conf에서 Saša Jurić의 강연은 코드 리뷰를 매력적인 이야기로 바꾸었습니다. 그는 크고 복잡한 풀 리퀘스트(PR)의 일반적인 문제점을 지적하고, 이것이 피상적인 리뷰, 보안 위험, 유지보수가 어려운 코드베이스로 이어지는 것을 강조했습니다. 중요한 점은 검토 가능한 PR은 이상적으로 5~10분 안에 검토할 수 있어야 하며, 코드 줄 수는 300줄 이하여야 한다는 것입니다. 각 변경 사항의 이유와 단계를 명확하게 설명하는 간결하고 이야기 형식의 커밋 메시지를 작성하여 이를 달성할 수 있습니다. 큰 기능을 작은 PR로 분할하고 `git fixup`과 같은 도구를 사용하여 깔끔한 커밋 기록을 유지하는 것은 효율적인 코드 리뷰와 더 높은 품질의 코드에 필수적입니다. 강연에서는 의미 없는 "LGTM"보다 "이해할 수 없습니다"라고 말하는 것이 더 낫다고 강조했습니다.