diff --git "a/\354\261\225\355\204\260_19/\353\263\200\354\210\230\353\257\270.md" "b/\354\261\225\355\204\260_19/\353\263\200\354\210\230\353\257\270.md" new file mode 100644 index 0000000..0c2a7f8 --- /dev/null +++ "b/\354\261\225\355\204\260_19/\353\263\200\354\210\230\353\257\270.md" @@ -0,0 +1,37 @@ +코드 리뷰는 소프트웨어 개발에서 없어서는 안되는 도구 +(코드 베이스의 가독성, 유지보수성 개선을 위해) + +## 코드 리뷰 도구 원칙 + +- **간결성** : 코드 리뷰를 쉽게 진행할 수 있도록 매끄러운 인터페이스를 제공 +- 신뢰 제공 : 코드 리뷰는 다른이의 일을 늦추는게 아닌 힘을 보태주는 행동이다. +- 익숙한 소통 방식 +- 워크플로 통합 + +> Critique, 구글에서 사용하는 코드리뷰도 저희가 하던 코드리뷰랑 크게 다르진 않네요. + +## 코드 리뷰 + +**1단계 - 변경 생성** +**2단계 - 리뷰 요청** +**3~4단계 : 변경 이해하고 댓글 달기** + +- ‘누구 차례 -> 리뷰어 등록과 어떤게 다른걸까요..? + **5단계 : 변경 승인** + 변경에 점수 매기기는 방식 +- **LGTM : 변경을 검토한 결과 우리 표준에 부합하며 미해결 댓글 해결 후 커밋해도 좋다** +- 승인 +- 미해결 댓글 개수 + **6단계 - 변경 커밋** +- 터미널에서 별도 명령을 실행할 필요없이 바로 커밋 가능 + +> 깃을 사용하면서 당연하게 해왔던것 같은데, 새삼 깃허브가 정말 잘 되어있다는게 느껴지네요. + +## 마치며 + +코드 리뷰에 쓰는 시간은 코드를 생산하지 못하는 시간이다. +-> 리뷰 프로세스를 최적화 하면 회사의 생산성이 개선될 수 있다. +-> 변경 리뷰에 드는 시간을 줄이려면 코드 리뷰 프로세스가 막힘없이 흘러야한다. + +결론적으로, +신뢰와 소통이 코드 리뷰의 프로세스의 핵심이다.