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
SOCAR
May 07, 2020
Programming
0
990
코드 리뷰 적응기
SOCAR
May 07, 2020
Tweet
Share
More Decks by SOCAR
See All by SOCAR
O'reilly Strata Data Conference New York 2019 후기
socar
2
1.5k
모빌리티데이터팀 신입 데이터 분석가의 1년 회고
socar
0
900
TF에서 팀 빌딩까지 9개월의 기록 : 성장하는 조직을 만드는 여정
socar
0
310
쏘카, 개발 - 쏘카 R&D 이야기
socar
0
520
MOBILITY x DATA : 모빌리티 산업의 도전 과제
socar
0
620
Other Decks in Programming
See All in Programming
JETLS.jl ─ A New Language Server for Julia
abap34
2
470
The Art of Re-Architecture - Droidcon India 2025
siddroid
0
150
Go コードベースの構成と AI コンテキスト定義
andpad
0
150
re:Invent 2025 のイケてるサービスを紹介する
maroon1st
0
160
AIの誤りが許されない業務システムにおいて“信頼されるAI” を目指す / building-trusted-ai-systems
yuya4
6
4.1k
C-Shared Buildで突破するAI Agent バックテストの壁
po3rin
0
420
Combinatorial Interview Problems with Backtracking Solutions - From Imperative Procedural Programming to Declarative Functional Programming - Part 2
philipschwarz
PRO
0
120
perlをWebAssembly上で動かすと何が嬉しいの??? / Where does Perl-on-Wasm actually make sense?
mackee
0
240
開発に寄りそう自動テストの実現
goyoki
2
1.6k
「コードは上から下へ読むのが一番」と思った時に、思い出してほしい話
panda728
PRO
39
26k
クラウドに依存しないS3を使った開発術
simesaba80
0
190
0→1 フロントエンド開発 Tips🚀 #レバテックMeetup
bengo4com
0
440
Featured
See All Featured
Dominate Local Search Results - an insider guide to GBP, reviews, and Local SEO
greggifford
PRO
0
24
Efficient Content Optimization with Google Search Console & Apps Script
katarinadahlin
PRO
0
260
Agile Leadership in an Agile Organization
kimpetersen
PRO
0
62
How to build a perfect <img>
jonoalderson
0
4.8k
Abbi's Birthday
coloredviolet
0
4k
Let's Do A Bunch of Simple Stuff to Make Websites Faster
chriscoyier
508
140k
実際に使うSQLの書き方 徹底解説 / pgcon21j-tutorial
soudai
PRO
196
70k
SEO Brein meetup: CTRL+C is not how to scale international SEO
lindahogenes
0
2.2k
brightonSEO & MeasureFest 2025 - Christian Goodrich - Winning strategies for Black Friday CRO & PPC
cargoodrich
2
71
Why Mistakes Are the Best Teachers: Turning Failure into a Pathway for Growth
auna
0
30
The Web Performance Landscape in 2024 [PerfNow 2024]
tammyeverts
12
990
Why You Should Never Use an ORM
jnunemaker
PRO
61
9.7k
Transcript
॑नथ যࢲ৬ ܻ٘࠭ח פয ѐߊ, ٘ ܻ࠭ ӝ
য়טফӝೡѪ 코드 리뷰 경험 없는 개발자의 코드 리뷰 적응기
ܻ٘࠭оޤ 코드리뷰 경험 전무 개인 프로젝트 팀 프로젝트
ܻ٘࠭അݾب
내 코드는 그냥 통과할 수 있을 것 같다는 막연한 자신감이
듦 ܻ٘࠭അݾب
PR 올리기
ղ13
ܻ13ӏ 개인 브랜치 네이밍 깃허브 마일스톤 ,라벨 세팅
ѐੋ࠳ے֎߁ CFBOBXFTPNFQSFTFOUBUJPO
݃ੌझహ ۄ߰ 마일스톤 프로젝트 전체 진행 상황 라벨 PR의 성격
명시
সٜয৳ 이슈 할당 TF 단위 작업 TF 단위 검증 작업
완료?
도저히 코드 리뷰를 통과하지 못하고 머지되지 않음 স৮ܐj 코드 리뷰까지
끝나야 작업 끝 거의 2주 no merge
코드 리뷰를 쉽게 통과하기 위해서 어떤 노력을 해야 할까?
ܻ٘࠭ܳాҗೞӝਤ೧ࢲj 설계 리뷰 문서 공유 관련 코드 새로운 구조
ܻ٘࠭ܳాҗೞӝਤ೧ࢲj PR을 최대한 짧게 유지한다. - 작업 중에 보이는 추가
작업은 하나의 PR에서 처리하지 말고, 별개의 PR로 올린다. - 보일러 플레이트 생성 코드는 별도의 브랜치로 날리고, 해당 브랜치을 베이스로 PR을 올린다.
ܻ٘࠭ܳాҗೞӝਤ೧ࢲj PR 올릴 때 코멘트에 작업 의도를 남기자. - PR
코멘트에 Markdown 이용 - 코드 코멘트 - 기획 문서, 디자인 링크 공유하기
참조가 될 만한 문서는 모두 링크를 건다. 세부 구현 아이디어에
대해서 설명한다.
코드 리뷰 보기
코드 작업, 코드 리뷰… 정신없다. ઁ աبܻ٘࠭ܳࠅٸ 코드 리뷰 안
되면, 릴리즈가 느려진다. 코드가 Merge 되기 위해서, 2명 이상의 reviewer의 approve가 필요하다.
코드 리뷰를 쉽게 보기 위해서 어떤 노력을 해야할까?
ܻ٘࠭ܳऔѱࠁӝਤ೧ࢲj Github의 가독성을 높이자. (feat. Chrome Extension)
܁झబ࣌ 0DUPUSFF 기존 깃 허브는 패키지 구조로 어떤 변화가 있는지
알기 어렵다. 유료 기능은 더 좋은게 많다고 합니다…
܁झబ࣌ (JUIVC%BSL5IFNF 다크 모드 덕후인 나는, 라이트 모드 Github가 눈
아프다.
ܻ٘࠭ܳऔѱࠁӝਤ೧ࢲj 실제로 코드를 받아오자. IDE 활용하기.
*%&ীࢲ٘ܳࠁݶ 코드 구조 파악 코드 구조 제안 동작성 확인
ܻ٘࠭ܳऔѱࠁӝਤ೧ࢲj 오프라인 코드 리뷰 회의를 진행하자. 온라인을 벗어나자.
য়ۄੋܻ٘࠭ ݽفझܽখীݽৈࢲܻ٘࠭ 온라인 한계 촉박한 일정 모든 팀원이 알아야
할 중요한 기능 집중도
ܻ٘࠭חޙച 매주 금요일 정기 회고를 통해 커뮤니케이션 PR 규칙 리뷰
활성화 더 효과적인 리뷰 환경
पӏೞӝ - 최대 허용 PR 10개 - 일주일에 한 번은
오프라인 코드 리뷰 회의 시간 가지기 - 본인 PR 1주 이상 묶여있을 시 작업 중지 - 슬랙 봇으로 온라인 리뷰를 안 한 사람에게 알람 보내기
Q&A 이메일:
[email protected]
감사합니다.