Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Sign up for free
Menu
Search
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Pricing
Search
Sign in
Sign up for free
코드 리뷰 적응기
Search
Sponsored
·
Your Podcast. Everywhere. Effortlessly.
Share. Educate. Inspire. Entertain. You do you. We'll handle the rest.
→
SOCAR
May 07, 2020
Programming
1k
0
Share
Embed
Copy iframe code
Copy JS code
Copy link
Start on current slide
코드 리뷰 적응기
SOCAR
May 07, 2020
More Decks by SOCAR
See All by SOCAR
O'reilly Strata Data Conference New York 2019 후기
socar
2
1.6k
모빌리티데이터팀 신입 데이터 분석가의 1년 회고
socar
0
920
TF에서 팀 빌딩까지 9개월의 기록 : 성장하는 조직을 만드는 여정
socar
0
330
쏘카, 개발 - 쏘카 R&D 이야기
socar
0
570
MOBILITY x DATA : 모빌리티 산업의 도전 과제
socar
0
650
Other Decks in Programming
See All in Programming
[Dev Group Paris Octobre 26] Agentforce, SIforce, Claudeforce, Slackbot… on s'y retrouve ?
nabondance
0
110
Everything will be SERVERLESS — 信じて運用した10年の経験値 / Everything Will be Serverless — Lessons Learned from 10 Years of Operational Experience
seike460
PRO
1
1.1k
Optunaを作ったときにやりたかったこと、その後 (2026/10/16, Optuna Meetup #3)
iwiwi
1
430
Augmenting AI with the Power of Jakarta EE
ivargrimstad
0
190
Railsのためのパスキー実践入門 〜マルチテナントSaaSでの意思決定〜
utakaha
2
560
Agentic Software Factoryに、すごく賢いIF文を。 / super smart IF statement into the Agentic Software Factory.
rkaga
8
2.2k
Agents on Rails - Rails at Scale 2026
irinanazarova
0
560
[2026-09-26]空論ジェネリックプロセス~テスト資産とAIで紡ぐ、再現可能なパフォーマンスチューニングの話~
tosite
0
270
UnityでSystem.Net.WebSocketsなWebSocketサーバが動かないのでUnity Monoのコードを覗いてみた / about implementing websocket server with unity mono
drumath2237
1
610
Next.js: The final boss of Web Performance
guaca
0
260
ジョジョの世界をXRで再現する
mofurune
0
160
dotNET_vulnclicker_intro.pdf
deka_morita
0
140
Featured
See All Featured
The Illustrated Guide to Node.js - THAT Conference 2024
reverentgeek
1
570
GraphQLとの向き合い方2022年版
quramy
50
15k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
37
6.6k
Testing 201, or: Great Expectations
jmmastey
46
8.3k
The B2B funnel & how to create a winning content strategy
katarinadahlin
PRO
1
570
The State of eCommerce SEO: How to Win in Today's Products SERPs - #SEOweek
aleyda
2
12k
Public Speaking Without Barfing On Your Shoes - THAT 2023
reverentgeek
1
600
Practical Orchestrator
shlominoach
192
12k
SERP Conf. Vienna - Web Accessibility: Optimizing for Inclusivity and SEO
sarafernandez
2
1.6k
Context Engineering - Making Every Token Count
addyosmani
9
1.2k
What does AI have to do with Human Rights?
axbom
PRO
1
2.5k
Efficient Content Optimization with Google Search Console & Apps Script
katarinadahlin
PRO
1
930
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]
감사합니다.