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
160
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
230
Educating Enfys (lightning talk)
thatdamnqa
0
190
There's More to Code Reviews than You Might Think ✩
thatdamnqa
0
93
There's More To Code Reviews Than You Might Think (lightning talk)
thatdamnqa
0
180
The Less Dull Bits of Testing
thatdamnqa
0
150
Other Decks in Programming
See All in Programming
Prepare for Jakarta EE 11 - Performance and Developer Productivity
ivargrimstad
0
720
StoreKit2によるiOSのアプリ内課金のリニューアル
kangnux
0
110
DMMプラットフォームがTiDB Cloudを採用した背景
pospome
8
4k
코틀린으로 멀티플랫폼 만들기
pangmoo
0
150
HUIT新歓2024「競技プログラミング、やってみませんか?」
slephy2784
1
270
単体テストを書かない技術 #phpcon_odawara
o0h
PRO
26
8.2k
Komplexe Oberflächen mit SVG und der Web Animation API
joergneumann
0
670
Java 22 Overview
kishida
1
180
VS Code をプロダクトにどう取り込むか
onomax
1
350
使ってみよう Azure AI Document Intelligence
kosmosebi
2
290
大規模Reactアプリのリアーキテクチャ~8万行のTanStack Query移行の軌跡~
kj455
4
940
What We Can Learn From OSS
inouehi
0
420
Featured
See All Featured
Side Projects
sachag
451
41k
Fontdeck: Realign not Redesign
paulrobertlloyd
76
4.9k
Product Roadmaps are Hard
iamctodd
44
9.7k
How to Create Impact in a Changing Tech Landscape [PerfNow 2023]
tammyeverts
14
1.5k
The Brand Is Dead. Long Live the Brand.
mthomps
49
28k
How GitHub (no longer) Works
holman
304
140k
Happy Clients
brianwarren
92
6.4k
A designer walks into a library…
pauljervisheath
200
23k
The Pragmatic Product Professional
lauravandoore
25
5.8k
For a Future-Friendly Web
brad_frost
172
9k
Infographics Made Easy
chrislema
238
18k
The Success of Rails: Ensuring Growth for the Next 100 Years
eileencodes
30
6k
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