Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
코드 리뷰어 활동 후기
Search
RIDI
February 10, 2017
Technology
3
890
코드 리뷰어 활동 후기
발표자: 뷰어팀 김광래
RIDI
February 10, 2017
Tweet
Share
More Decks by RIDI
See All by RIDI
원격 근무 팀 운영 경험 공유
ridi
0
2.1k
SVG Icon Design Guide
ridi
2
3.1k
빠르게 훑어보는 리디페이 백엔드 개발기
ridi
2
4.8k
Next.js는 정말 zero config였다.
ridi
0
1.9k
3일 걸릴 것 같던 구매목록 다운로드는 왜 3주가 걸렸을까?
ridi
0
330
원격으로 한 달 일해보니
ridi
0
1.4k
리디북스 앱에 S Pen Remote 연동하기
ridi
2
2.5k
UI 라이브러리 개발기
ridi
1
2.3k
테스트 환경 개선하기
ridi
8
3.2k
Other Decks in Technology
See All in Technology
ペパボのオブザーバビリティ研修2024 説明資料
kesompochy
0
1.1k
dxd2024-生成AIに振り回された3か月間の成功と失敗/dxd2024-link-and-motivation
lmi
2
260
開発と事業を繋ぐ!SREのオブザーバビリティ戦略 ~ Developers Summit 2024 Summer ~
leveragestech
0
640
Classmethod Odyssey 登壇資料
yamahiro
0
390
「我々はどこに向かっているのか」を問い続けるための仕組みづくり / Establishing a System for Continuous Inquiry about where we are
daitasu
0
170
累計ダウンロード数1億8000万を超えるアプリケーションプラットフォームのレガシーシステム脱却とモダン化への道
kmitsuhashi
0
120
サーバーレスAPI(API Gateway+Lambda)とNext.jsで 個人ブログを作ろう!
shuntaka
PRO
0
560
How to Think Like a Performance Engineer
csswizardry
4
590
LINE WORKSへ簡単通知!Incoming Webhookアプリの紹介
mmclsntr
0
110
Azure Pipelinesを使用したCICDベースラインアーキテクチャ実践
yuriemori
0
190
楽しくGoを学び合う、LayerXの勉強会文化 / LayerX's study culture of having fun and learning Go together
ar_tama
2
350
たくさん本を読んだけど 1年後には綺麗サッパリ!を乗り越えて 学習の鬼になるぞ👹
yum3
0
160
Featured
See All Featured
The World Runs on Bad Software
bkeepers
PRO
63
11k
Scaling GitHub
holman
458
140k
Optimizing for Happiness
mojombo
373
69k
Fashionably flexible responsive web design (full day workshop)
malarkey
399
65k
Agile that works and the tools we love
rasmusluckow
325
20k
Build your cross-platform service in a week with App Engine
jlugia
227
17k
Typedesign – Prime Four
hannesfritz
37
2.2k
Code Reviewing Like a Champion
maltzj
517
39k
Learning to Love Humans: Emotional Interface Design
aarron
269
39k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
277
13k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
17
8.7k
Building Your Own Lightsaber
phodgson
101
5.9k
Transcript
코드 리뷰어 활동 후기 뷰어팀 김광래, 17.2.10
저에 대하여
근래에 코드 리뷰어 역할을 자주 맡게 되면서 제가 했던 /
하고 있는 고민들을 공유하고자 합니다!
코드 리뷰 : 개발 결과에 대한 의견을 교환하는 과정
코드 리뷰에서 확인해야할 것 - 잘 동작하는가? - ‘좋은 코드’인가?
- 스타일의 일관성 - 간결 + 이해하기 쉬움 - 적절히 분리 + 유연한 구조 - 테스트하기 쉬움
고민 1 : 좋은 도구? GitLab - Upsource - GitHub
Upsource -빠름 -특정 commit 제외 가능 -Comment 완료 처리 가능
https://d3nmt5vlzunoa1.cloudfront.net/wp-content/uploads/2013/09/image_thumb2.png
GitHub -Issue / review의 관리 및 연계가 쉬움 -Commit 선택에
제약↑ -다소 느림
GitLab -GitHub과 유사 -매우 느림 -Comment가 많으면 오류 발생 https://i.ytimg.com/vi/aWFMxYT7n6Y/maxresdefault.jpg
고민 2 : 좋은 Process? 좋은 결과 + 빠른 진행
속도
Remind : 코드 리뷰에서 확인해야할 것 - 잘 동작하는가? -
‘좋은 코드’인가? - 스타일의 일관성 - 간결 + 이해하기 쉬움 - 적절히 분리 + 유연한 구조 - 테스트하기 쉬움
무엇을 하는 코드인지 파악하기 https://www.entropywins.wtf/blog/wp-content/uploads/2014/04/cr.png
잘 동작하는지 확인하기 – 좋았던 점 - 리뷰 전 Task
파악에 유용 - 오류 Double check 가능 - “버그로 인한 코드 변경 리뷰 시간 증가” 예방
잘 동작하는지 확인하기 – 시행착오 오류를 발견했을 때 … -
공유 시점 - 공유 대상 - 공유 수단 : 구두 / Slack / Asana
잘 동작하는지 확인하기 – 문제점 - 코드 리뷰에 속하는 부분인지?
- Test 자동화로 생략 가능한 부분이 아닌지? - 디자인 확인 시에 체크해야 할 부분이 아닌지?
좋은 코드인지 확인하기 큰 변화를 가져올 수 있는 부분부터 -
전체적인 구조 디테일한 부분 - 리뷰의 부담을 줄이기 위해 http://m.memegen.com/ud0t0y.jpg
좋은 코드인지 확인하기 - 아직 다뤄보지 못한 부분을 리뷰할 때
- 더 좋은 구조 / 구현에 대한 고민이 있을 때 더 많은 질문 + 의견 교환
최대한 자동화하기 정량적인 체크 자동화 리뷰 부담 감소 -
Lint : 코드 스타일 확인, 잠재적 오류 검사 - CI : 빌드 성공 / Test, Lint 통과 여부 확인
리뷰 중 코드 변경에 대처하기 This page was outdated… click
here to refresh… 리뷰 범위 기억하기
- 해당 feature와는 연관성이 적지만 중요한 문제 - 리뷰 시
발견한 문제가 다른 부분에도 있는 경우 별도 Issue / Task 분리 리뷰 범위 밖의 문제를 발견한 경우
여전히 고민 중인 부분
리뷰 분량 문제 - 양이 많은 경우 너무 오랜 시간이
걸림 - 프로젝트를 작게 분리 디자인 확인 + 리뷰 반복 - 디자인 확인 일정도 같이 변경되어야 !
어디까지 Comment를 남겨야 하는지 - 작성자의 의도 : ? -
작성자의 취향 : ? - 관련 History : ? 궁금하면 물어보기 https://i.stack.imgur.com/dLTF8.png
좋은 의사 표현 방식은 무엇인지 - 상대가 쉽게 이해할 수
있는 표현 - 글 vs 말 - 스크린샷 첨부 기능 등 이용 - 상대의 감정을 상하지 않게 하는 표현
감사합니다!