<어떻게 코드 리뷰어가 당신을 사랑하게 할 수 | 커리어리

<어떻게 코드 리뷰어가 당신을 사랑하게 할 수 있을까?> 조금 자극적인 제목이지만, 좋은 코드 리뷰를 "받기" 위해 필요한 점에 대한 글입니다. 많은 팀에서 코드 리뷰 문화가 정착되어 있지만, 자신의 시간을 쪼개어 다른 사람의 코드를 본다는 것은 결코 쉬운 일은 아닙니다. 형식적으로 리뷰를 할 때도 있고, 요청자의 자존심을 고려해 충분한 피드백을 주지 못할 때도 있고, 오히려 공격적인 피드백으로 감정적인 다툼이 일어날 때도 있습니다. 좋은 코드 리뷰를 받기 위한 황금률이 있습니다. - 리뷰어의 시간을 소중하게 생각하라. - 리뷰어가 코드의 흥미로운 부분에 집중하여 좋은 피드백을 생성할 수 있도록 하라. - 방어적인 태도를 버리고 신중하게 커뮤니케이션 하라. 여기서 코드 리뷰를 원활하게 할 수 있는 13가지의 테크닉을 소개합니다. 1. 리뷰 요청 전에 먼저 스스로의 코드를 한 번 더 검토해 보라. 2. 변경 목록은 명확하게 상술할 필요가 있다. 3. 컨벤션이나 테스트 실패 등의 하찮은 문제로 리뷰어의 시간을 낭비하지 않도록 리뷰 요청 전에 자동화된 컨벤션 체크와 유닛 테스트를 거치도록 하라. 4. 만약 리뷰어가 코드의 의도가 모호하다고 지적한다면, 리뷰에 답변을 다는데 그치지 말고 그 자체로 충분한 설명이 될 수 있는 명확한 코드로 수정하라. 5. 변경 목록의 하나의 변경 사항(commit)은 하나의 문제만을 해결하는 내용을 담을 수 있도록 하라. 6. 기능적 변경과 비기능적 변경(ex. 리팩토링, 인덴트 등의 포맷 변경 등)을 분리하라. 리팩토링 시에는 동작의 무결성을 보장하기 위한 테스트 코드와 함께 리뷰를 요청하라. 7. 지나치게 큰 변경 사항을 한 목록에 담지 말고 작은 변경 단위로 나누어서 리뷰어의 부담을 덜어주도록 하라. 8. 리뷰에 방어적으로 대응하지 말고 정중하게 응답하라. 9. 가끔 리뷰어가 잘못된 리뷰를 할 때도 있다. 리뷰어 뿐만 아니라 다른 사람도 착각할 수 있는 모호함이 있는지 확인하고, 보다 명확한 코드로 수정하거나 주석을 첨부하라. 10. 자신의 작업 상태를 명확하게 커뮤니케이션할 수 있도록 하라. 리뷰의 결과를 반영하고 있다거나 추가 작업 중이라는 것을 리뷰어에게 명확하게 알려서 필요없는 리뷰나 작업을 하지 않는 상황이 될 수 있도록 하라. 11. 또한 리뷰어도 모호한 리뷰를 할 때가 있다. 여기에 예민한 반응을 하기 보다 추가적인 정보를 얻을 수 있도록 정중하게 요청하라. (ex. 어떤 변화가 도움이 될까요?) 12. 작성자와 리뷰어의 의견 중 어떤 것을 취해도 큰 문제가 없을 때, 코드를 읽는 사람의 의견을 고려하여 리뷰어에게 우선권을 주도록 하라. 13. 리뷰에 대해 빠른 피드백을 전달하여 리뷰어의 컨텍스트가 유실되지 않도록 주의하라. --- (🚀 채용 광고) '신상마켓'을 운영하는 딜리셔스에서는 동대문 패션 생태계를 기술을 통하여 더 나은 모습으로 디지털화하고 함께 성장하여 글로벌까지 연결하는 꿈을 갖고 있습니다.🚀 이런 분들을 찾고 있습니다. - Java / RubyOnRails Developers - Android / iOS Developers - Web Developers (React.js / Vue.js) - Data Engineers on AWS - DevOps on AWS - Product Managers / Product Designers - Data Analysists - Ad Platform Developers / PMs b2b 뿐만 아니라 b2b2c/글로벌 등 다양한 제품군을 준비 중이고, 주도적으로 오너십을 갖고, 자율적으로 일하는 환경을 지향합니다 ☺️ 채용 더 알아보기 >> https://www.wanted.co.kr/company/3059

How to Make Your Code Reviewer Fall in Love with You

Mtlynch

2021년 2월 23일 오전 4:17

댓글 1

함께 보면 더 좋은

(딜리셔스 기술블로그의 라이벌;;;) 컬리 기술블로그에 실린 카트 개발기입니다. 커머스 플랫폼에서 탐색-주문 과정의 중간에 위치한 장바구니는 쇼핑의 필수 요소라고 할 수 있으며, 이에 따라 복잡한 비즈니스 로직을 가지기 마련입니다. 하나의 작은 서비스에 대해 계속 고민하고 차근차근 문제를 해결해 나가는 모습이 멋집니다. —- '신상마켓'을 운영하는 딜리셔스에서는 동대문 패션 생태계를 기술을 통하여 더 나은 모습으로 디지털화하고, 함께 성장하여 글로벌까지 연결하는 꿈을 갖고 있습니다. B2B 뿐만 아니라 풀필먼트 및 글로벌향의 다양한 제품을 개발/운영 중이며, 주도적으로 오너십을 갖고 자율적으로 일하는 환경을 지향합니다. 저희 팀과 함께 할 실력 있는 분들을 적극적으로 모시고 있습니다. 많은 지원 부탁드려요! 🚀 딜리셔스 채용 더 알아보기 >> https://www.notion.so/e8747fb16efe4f1ba7747e7f96f9d373 👩‍💻 딜리셔스 개발팀 소개 >> https://dealicious-inc.github.io/2021/06/14/dev-team-chronicle.html

카트 개발 연대기

Kurly

추천 프로필

현직자에게 업계 주요 소식을 받아보세요.

현직자들의 '진짜 인사이트'가 담긴 업계 주요 소식을 받아보세요.

커리어리 | 일잘러들의 커리어 SNS