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
Zen Code Reviews
Search
Xavier F. Gouchet
PRO
April 20, 2017
Video
Programming
0
1
Zen Code Reviews
A lightning talk on how to do code reviews without getting into a conflict with coworkers…
Xavier F. Gouchet
PRO
April 20, 2017
Tweet
Share
Video
More Decks by Xavier F. Gouchet
See All by Xavier F. Gouchet
Being an ethical software engineer
xgouchet
PRO
0
230
Libérez votre créativité
xgouchet
PRO
0
97
Unleash your Programming Creativity
xgouchet
PRO
0
150
Demystifying the Test Pyramid
xgouchet
PRO
1
460
Successfully Publishing a Tested, Upgradeable and Documented Open Source Library
xgouchet
PRO
0
86
Property Based Testing in Practice
xgouchet
PRO
0
65
Develop your CI tools
xgouchet
PRO
2
390
How to write a safe and reliable Open Source Library
xgouchet
PRO
1
260
Benchmarking your app's performance
xgouchet
PRO
0
210
Other Decks in Programming
See All in Programming
Cursor/Devin全社導入の理想と現実
saitoryc
28
21k
20250429 - CNTUG Meetup #67 / DevOps Taiwan Meetup #69 - Deep Dive into Tetragon: Building Runtime Security and Observability with eBPF
tico88612
0
170
AI時代の開発者評価について
ayumuu
0
230
Optimizing JRuby 10
headius
0
570
今話題のMCPサーバーをFastAPIでサッと作ってみた
yuukis
0
110
Cursorを活用したAIプログラミングについて 入門
rect
0
160
Making TCPSocket.new "Happy"!
coe401_
1
3.1k
State of Namespace
tagomoris
5
2.4k
スモールスタートで始めるためのLambda×モノリス(Lambdalith)
akihisaikeda
2
360
Instrumentsを使用した アプリのパフォーマンス向上方法
hinakko
0
230
複雑なフォームの jotai 設計 / Designing jotai(state) for Complex Forms #layerx_frontend
izumin5210
6
1.5k
Road to RubyKaigi: Making Tinny Chiptunes with Ruby
makicamel
4
540
Featured
See All Featured
Fontdeck: Realign not Redesign
paulrobertlloyd
84
5.5k
The Success of Rails: Ensuring Growth for the Next 100 Years
eileencodes
45
7.2k
Speed Design
sergeychernyshev
29
930
What's in a price? How to price your products and services
michaelherold
245
12k
A Modern Web Designer's Workflow
chriscoyier
693
190k
Writing Fast Ruby
sferik
628
61k
Stop Working from a Prison Cell
hatefulcrawdad
268
20k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
32
5.5k
Sharpening the Axe: The Primacy of Toolmaking
bcantrill
41
2.3k
KATA
mclloyd
29
14k
Documentation Writing (for coders)
carmenintech
71
4.7k
Faster Mobile Websites
deanohume
306
31k
Transcript
Zen Code Reviews The (fast) way towards painless code reviews
CodeMobile UK 2017 - Chester
Why ◎ Find defects early ◎ Harmonize code base ◎
Generate discussion ◎ Share knowledge ◎ Team Building @xgouchet
◎ Pre vs Post Commit ◎ Mandatory vs Optional ◎
Remote vs Pair Review ◎ Static Analysis Workflows @xgouchet
Reviewable Code ◎ Keep the commits short ◎ git add
--patch ◎ Use a proper commit message ◦ CR Brief ◎ Review your own code @xgouchet
Commenting on issues ◎ Be precise ◦ “This uses too
much memory…” ◎ Argument ◦ “… because this object is not retained when …” ◎ Help fix it ◦ “You could instead use …” ◎ Define criticity ◦ “Itʼs not critical and can be done in a later commit” @xgouchet This is bad This is bad
Best behavior ◎ Stay open minded ◎ Make it about
the code ◎ Share knowledge ◎ Give as much as you receive @xgouchet