개발문화 2

A guide for Code Reviewers

해당 문서는 구글소스를 기반으로 작성되었습니다. https://chromium.googlesource.com/chromium/src/+/refs/heads/main/docs/cr_respect.md DO Assume competence & goodwill (능력과 선의를 가정하라) 비록 그들이 틀렸을지라도, 대부분 정보의 부재에서 오는 틀림이지 능력의 부재에서 오는 틀림이 아니다. Discuss in person (직접 토론하라) (코드리뷰 도구를 통한 경우) 의견불일치가 있을 시, 화상미팅/대면 등 직접 토론하라 Explain why (이유를 설명하라) 몇몇 코드는 틀렸다는 사실이 명확하더라도, 작성자가 작성하고자 했던 방향이 아닐 수 있다. 그러므로 "이것은 틀렸습니다." 라고 말하는 대신, 어떤 방..

DEVELOPMENT/DX&DC 2023.10.06

Software Engineering at Google : CHAPTER 9

본 포스팅은 "Software Engineering At Google"을 읽고 작성되었습니다. 출처: Software Engineering At Google - Chapter 9 Chapter 9 : Code Review Code Review Flow 작성자는 변경점을 작성하고, 스냅샷을 생성하여 코드리뷰 툴에 업로드한다. 작성자는 자동 검토 주석 등을 활용하여 사전 검토 후, 변경 사항을 검토자에게 메일로 전송하여 검토자가 확인 후 댓글을 달 수 있도록 요청한다. 검토자는 코드리뷰 툴에서 변경사항을 확인 후, 의견을 게시한다. 의견은 명시적 해결을 요청하거나, 단순 정보 제공일 수 있다. 작성자는 변경 사항을 수정하고 피드백을 기반으로 새 스냅샷을 업로드한 뒤, 검토자에게 다시 응답한다. (3,4 단계..

DEVELOPMENT/DX&DC 2023.10.06