셩잇님의 잡동사니

코드 리뷰는 조직의 상황에 따라 실행하는 목적도 방식도 목표도 다르다. 그렇기에 정답이 없다. 그래서 내가 코드리뷰에 관한 글을 시작하기 앞서 내가 처한 상황을 설명하지 않는다면, 읽는 사람도 글을 이해하기 쉽지 않을 것이다. 나와 함께 일하는 조직은 아래와 같았다. * 열 명 이하로 구성된 개발 조직이었다. * 대부분의 개발자가 신입 사원이거나 경력이 2년이 채 되지 않는 주니어 개발자였다. * 간혹 10년~20년 경력의 시니어 개발자가 있었으나 코드 리뷰 경험이 없었다. * 실력 있는 개발자를 채용하기 어려웠다. * Git을 SVN(Subversion)처럼 쓰고 있었다. * 이슈 기반의 개발을 하고 있지 않았다. 코드 리뷰어로서 내 목표는 세 가지였다. * 최소한의 코드 품질 유지 * 개발 코칭을 통..

요약 1. 로직을 이해하기 쉽도록 구현했는지를 위주로 코드리뷰합니다. 2. 코드 작성자는 확신이 없는 부분에 대해서 사전에 설명하는 코멘트를 남겨두면 좋습니다. 3. 코드 리뷰어는 자신의 코멘트가 필수적으로 반영되어야 하는지, 선택적으로 반영하면 좋은 것인지 명시하면 좋습니다. 4. 구글 등 다른 코드리뷰 문화도 참고하시면 좋습니다. 경험상 사람마다 코드리뷰에서 주로 보는 부분이 다르겠지만, 저는 로직을 주로 봅니다. 구현하고자 하는 것에 비해서 복잡하게 되어있지는 않은지, 맥락을 알아야지만 이해할 수 있도록 구현되어 있지는 않은지 점검하는 편이에요. 그래서 사실 코드를 읽다가 이해하기 어렵다 싶으면 그때부터 로직에 개선점이 있는지 주의 깊게 봅니다. 코드 스타일 등 린터나 포매터로 자동화할 수 있는 부..

요즘 매일 보는 앱 중에 깃허브(github)가 있습니다. 개발에서 손을 뗀 지 벌써 13년이 지났지만, SaaS 기업으로 회사를 키우는 입장에서 코드를 제품(product)을 구성하는 중요한 요소로 보고 있기 때문에 깃허브의 알람을 빠짐 없이 읽습니다. 코드를 중심으로 한 개발자들의 대화를 읽다 보면 코드를 짜지 않더라도 동료들이들이 무슨 생각을 하고, 어떤 논의들이 일의 중심에 놓이는지 큰 노력 없이 알 수 있어 무척 편리합니다. 이 글에서는 조직의 리더 입장에서 바라 본 코드 리뷰에 관한 경험을 토대로, 코드 리뷰가 인간적인 개발 문화를 만드는 데 얼마나 큰 기여를 하는지 말하고자 합니다. 출처 및 더 읽어보기 : 요즘 IT '안영희'님 https://yozm.wishket.com/magazine/..

🕊️ 왜 코드 리뷰를 해야 하는가? 우리가 코드 리뷰를 해야 하는 이유, 그리고 어떻게 하는 지에 대한 글을 가져왔습니다. 이번에는 '왜' 코드 리뷰를 해야 하는지, 그 이유를 알아봅시다. 코드 리뷰를 하는 5가지 방법 - [ ] 1️⃣ 내가 아닌 다른 사람이 나의 코드를 이해하는지 알 수 있다. 2️⃣ 혼잡한 변수 이름, 클래스, 메서드 등을 명확하게 할 수 있다. 3️⃣ 잊고 있는 것이 없는지 체크할 수 있다. 4️⃣ 같은 문제에 대해 더 나은 해결 방법이 있는지 동료에게 배울 수 있다. 5️⃣ 오타를 고칠 수 있다. 🐣 다음에 Pull Request를 날릴 때는, 먼저 당신의 코드를 스스로 리뷰 해보세요. 출처 : 커리어리 '조서희'님 https://careerly.co.kr/comments/809..