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
Reviewing Code Reviews
Search
Gio Lodi
August 30, 2018
Programming
1
1.1k
Reviewing Code Reviews
A look at the value of code reviews, with tips for authors, reviewers, and automation.
Gio Lodi
August 30, 2018
Tweet
Share
More Decks by Gio Lodi
See All by Gio Lodi
WWDC 2021: What's New in Testing
mokagio
0
1.8k
Slack Considered Harmful
mokagio
0
120
Less Conventional Swift (Lighting Talk)
mokagio
0
81
Talking myself into the value of acceptance testing
mokagio
0
46
Other Decks in Programming
See All in Programming
shadcn/uiを使ってReactでの開発を加速させよう!
lef237
0
380
Java Webフレームワークの現状 / java web framework at burikaigi
kishida
7
1.9k
[Fin-JAWS 第38回 ~re:Invent 2024 金融re:Cap~]FaultInjectionServiceアップデート@pre:Invent2024
shintaro_fukatsu
0
340
ErdMap: Thinking about a map for Rails applications
makicamel
1
1.1k
Package Traits
ikesyo
2
230
バックエンドのためのアプリ内課金入門 (サブスク編)
qnighy
7
1.4k
Запуск 1С:УХ в крупном энтерпрайзе: мечта и реальность ПМа
lamodatech
0
980
AWSマネコンに複数のアカウントで入れるようになりました
yuhta28
2
150
Flatt Security XSS Challenge 解答・解説
flatt_security
0
1k
SpringBoot3.4の構造化ログ #kanjava
irof
2
750
watsonx.ai Dojo #6 継続的なAIアプリ開発と展開
oniak3ibm
PRO
0
260
Simple組み合わせ村から大都会Railsにやってきた俺は / Coming to Rails from the Simple
moznion
3
3.7k
Featured
See All Featured
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
28
4.5k
Improving Core Web Vitals using Speculation Rules API
sergeychernyshev
6
220
GitHub's CSS Performance
jonrohan
1030
460k
Product Roadmaps are Hard
iamctodd
PRO
50
11k
JavaScript: Past, Present, and Future - NDC Porto 2020
reverentgeek
47
5.1k
The Art of Programming - Codeland 2020
erikaheidi
53
13k
Designing on Purpose - Digital PM Summit 2013
jponch
117
7.1k
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
jcasabona
30
2.1k
Done Done
chrislema
182
16k
Art, The Web, and Tiny UX
lynnandtonic
298
20k
Gamification - CAS2011
davidbonilla
80
5.1k
Scaling GitHub
holman
459
140k
Transcript
Reviewing Code Reviews
Gio iOS Lead at @mokagio gio.codes
What & Why
Code Reviews are… Discussion on the code
Code Reviews are… Learning opportunity
Code Reviews are… ✍ share new things they learnt "
teach what they know
Code Reviews are… Growth opportunity for the team
Invest in Code Reviews because… If code reviews make the
team better, then better code reviews will make the team even better
“Indeed, the ratio of time spent reading versus writing is
well over 10 to 1. We are constantly reading old code as part of the effort to write new code. Robert C. Martin (Uncle Bob) - Clean Code
"" "" "" "" "" ✍
Invest in Code Reviews because… Less back and forth
1. As an author ✍ 2. As a reviewer "
3. As a robot #
As an author ✍
As an author ✍ Write a great PR description -
What AND Why - Context - Graphs, Images, GIFs, Links
None
None
None
As an author ✍ Review your PRs yourself first
As an author ✍ Keep your PRs small and focused
https://twitter.com/iamdevloper/status/397664295875805184
As a reviewer "
As a reviewer " Watch your language
None
As a reviewer " $ “How would I feel if
someone left this comment to me?”
As a reviewer " Ask questions
As a reviewer " Constructive Criticism
As a reviewer " Strong rationales (and links to back
them up)
None
As a reviewer " Review what the code DOES not
how it LOOKS
As a reviewer " Use a style guide
https://github.com/iflix-letsplay/swift-style-guide
As a reviewer " You can code review your style
guide %
As a robot #
As a robot # Automate your style guide Linters Formatters
https://github.com/realm/SwiftLint
https://houndci.com/
https://prettier.io/
As a robot # Check for mistakes easy to make
https://danger.systems
https://danger.systems
https://danger.systems
As a robot # Use a PR template
https://blog.github.com/2016-02-17-issue-and-pull-request-templates/
https://blog.github.com/2016-02-17-issue-and-pull-request-templates/
https://twitter.com/mokagio/status/1019709696872087557
Recap ☝ Code Reviews are first and foremost a growth
opportunity for the team ' A good Code Review starts with the author ✍ As a reviewers, be inquisitive and constructive " Use automation to remove friction and take care of mundane checks #
Questions❓
@mokagio gio.codes/talks/lastconf18