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
There's more to code review than you might think
Search
Daniel Shaw
June 11, 2016
Programming
0
230
There's more to code review than you might think
Lightning talk, given at PHPSC 2016
Daniel Shaw
June 11, 2016
Tweet
Share
More Decks by Daniel Shaw
See All by Daniel Shaw
There's more to code review than you might think
thatdamnqa
1
290
Educating Enfys (lightning talk)
thatdamnqa
0
240
There's More to Code Reviews than You Might Think ✩
thatdamnqa
0
100
There's More To Code Reviews Than You Might Think (lightning talk)
thatdamnqa
0
240
The Less Dull Bits of Testing
thatdamnqa
0
220
Other Decks in Programming
See All in Programming
Introduction to kotlinx.rpc
arawn
0
700
SwiftUI Viewの責務分離
elmetal
PRO
1
240
2024年のkintone API振り返りと2025年 / kintone API look back in 2024
tasshi
0
220
dbt Pythonモデルで実現するSnowflake活用術
trsnium
0
150
2024年のWebフロントエンドのふりかえりと2025年
sakito
2
250
Java Webフレームワークの現状 / java web framework at burikaigi
kishida
9
2.2k
『テスト書いた方が開発が早いじゃん』を解き明かす #phpcon_nagoya
o0h
PRO
2
210
富山発の個人開発サービスで日本中の学校の業務を改善した話
krpk1900
4
390
Grafana Loki によるサーバログのコスト削減
mot_techtalk
1
130
クリーンアーキテクチャから見る依存の向きの大切さ
shimabox
2
280
1年目の私に伝えたい!テストコードを怖がらなくなるためのヒント/Tips for not being afraid of test code
push_gawa
0
140
GoとPHPのインターフェイスの違い
shimabox
2
190
Featured
See All Featured
A designer walks into a library…
pauljervisheath
205
24k
How GitHub (no longer) Works
holman
314
140k
Done Done
chrislema
182
16k
Scaling GitHub
holman
459
140k
Art, The Web, and Tiny UX
lynnandtonic
298
20k
Music & Morning Musume
bryan
46
6.3k
Java REST API Framework Comparison - PWX 2021
mraible
28
8.4k
Building Applications with DynamoDB
mza
93
6.2k
Docker and Python
trallard
44
3.3k
Put a Button on it: Removing Barriers to Going Fast.
kastner
60
3.7k
Imperfection Machines: The Place of Print at Facebook
scottboms
267
13k
GitHub's CSS Performance
jonrohan
1030
460k
Transcript
THERE’S MORE TO CODE REVIEW THAN YOU MIGHT THINK Clair
Shaw @clairs · clairshaw.co.uk
What are Code reviews for?
Code reviews are not an indication of anybody's abilities
Check code style
Check code style Pick a house style, use it, communicate
it
Check code style Pick a house style, use it, communicate
it
Check code style Don't get too distracted by checking code
style
Check code style Automate if you can. Bad news can
be better received by a cruel and lifeless script
Review the configuration
Review the configuration { "require": { "usefultool/tool": "1.1" } }
Review the configuration { "require": { "usefultool/tool": "1.2.1" } }
Review the configuration { "require": { "usefultool/tool": "^1.2" } }
Check forwards compatibility
Check forwards compatibility PHP 5.5.32 PHP 5.6.18 PHP 7.0.3
Check forwards compatibility jQuery 1.12.1 jQuery 2.2.1
Review the documentation
Review the commit message
Review the commit message Update usefultool
Review the commit message [MYCOOLAPP-1234] Update UsefulTool lib Wider company
policy dictates that the latest UsefulTool v1 should be used. Update Composer config to reflect this.
Double check the code
Double check the code if (in_array($input, $expected)) { $this->showRestrictedSecrets(); }
else { $this->goAway(); }
Double check the code $expected = ['2', '4', '5', 'cake'];
$input = 0; if (in_array($input, $expected)) { echo "I'm in!"; }
Double check the tests
Final thoughts…
Final thoughts… All output should be reviewed. It’s not personal
Final thoughts… Often, a peer review can teach something to
two people
Thanks. Feedback please! https://joind.in/talk/29752
[email protected]
@clairs