두 번째

• 뱅크샐러드 코드리뷰 문화 전달

• PR 리뷰 팁

• 커밋에 대하여

뱅크샐러드 코드리뷰 문화 전달

코드 리뷰 in 뱅크샐러드 개발 문화 | 뱅크샐러드

안녕하세요, 뱅크샐러드 BanksaladX iOS Engineer 정진규입니다. 코드 리뷰란 개발자가 작성한 코드를 다른 사람들이 검토하고 피드백을 전달하며, 다시 작성자가 반영하는 과정을 말합니다. 이 과정을 통해 서비스의 기술적인 구현 부분에 대해 공유·습득할 수 있으며, 일관된 아키텍처를 유지하고, 문제를 해결할 수 있는 새로운 관점을 발견할 수도 있습니다.

https://blog.banksalad.com/tech/banksalad-code-review-culture/

• 우선순위를 표기하여서, 명확한 커뮤니케이션

• just ask; 형태로 표현하여 질문

• 깃헙기능과 연관하여 진행해도 무관
• p1, p2 → Request Change
• p3 → comment
• p4, p5 → Approve

PR 리뷰 팁

• 깃헙의 PR기능들 - comment, approve, request change

• 자료와 함께 코드리뷰하는 방법 - 리뷰에 공신력이 생긴다.
• e.g. 읽기 쉽게(readable) 코드를 작성하는 팁을 함께 공유
• 마틴 파울러의 리팩토링 팁 공유
Catalog of Refactorings
I've put together this catalog to help you find and explore the refactorings the 2nd edition. Each card shows the name used in the second edition together with any aliases for the refactoring (such as names for first edition refactorings that it replaces). The card links to the refactoring's online page.

https://refactoring.com/catalog/

• 온라인 PR에서 리뷰하기 힘든 경우
• 로컬에서 pull 받은 후 해당 branch에서 보는것도 방법!

커밋에 대하여

• 상세한 커밋!
• 커밋은 설명이 상세할수록 좋다.
• 2줄이상 작성시 상세하게 작성 가능

• 커밋단위 리뷰
• 커밋단위 리뷰가 가능하도록, 커밋을 하나의 task단위로 끊어서 하는 것도 방법이다.
• 단계별로 리뷰가 필요할때 or 파일이 많을때

• 커밋prefix conventional commit에 대한 추가설명
Conventional Commits
커밋 메세지에 사용자와 기계 모두가 이해할 수 있는 의미를 부여하기 위한 스펙

https://www.conventionalcommits.org/ko/v1.0.0/
• 오픈소스에 자주 사용된다.
• 추가 설명: sementic version
1. conventional commit 룰에 맞게 커밋
2. 이런 솔루션을 사용하면, merge 시점에 아래 사항이 자동 진행
1. commit들을 파싱
2. sementic version중 어떤 버전을 올려야할지 판단
1. feat: minor버전
2. feat이외의 prefix: patch
3. BREAKING CHANGE 커밋: major버전
3. 이전버전과 이후 버전 사이의 변경사항이 적혀있는 changeLog 문서 generate

• 다른 사람의 코드가 익숙하지 않아서 일수도

• 온라인 플랫폼에서 보는 것이 익숙하지 않아서 일수도

• …

• 로컬에서 본인이 익숙한 에디터에서 확인하거나

• 깃헙 desktop 앱에서보는 방법

• …