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.2k
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.9k
Slack Considered Harmful
mokagio
0
120
Less Conventional Swift (Lighting Talk)
mokagio
0
96
Talking myself into the value of acceptance testing
mokagio
0
65
Other Decks in Programming
See All in Programming
The Evolution of Enterprise Java with Jakarta EE 11 and Beyond
ivargrimstad
0
550
The Niche of CDK Grant オブジェクトって何者?/the-niche-of-cdk-what-isgrant-object
hassaku63
1
720
テストから始めるAgentic Coding 〜Claude Codeと共に行うTDD〜 / Agentic Coding starts with testing
rkaga
17
6.2k
副作用と戦う PHP リファクタリング ─ ドメインイベントでビジネスロジックを解きほぐす
kajitack
3
480
AI時代の『改訂新版 良いコード/悪いコードで学ぶ設計入門』 / ai-good-code-bad-code
minodriven
24
10k
マッチングアプリにおけるフリックUIで苦労したこと
yuheiito
0
250
バイブコーディング超えてバイブデプロイ〜CloudflareMCPで実現する、未来のアプリケーションデリバリー〜
azukiazusa1
2
730
ZeroETLで始めるDynamoDBとS3の連携
afooooil
0
130
NEWT Backend Evolution
xpromx
1
160
プロダクトという一杯を作る - プロダクトチームが味の責任を持つまでの煮込み奮闘記
hiliteeternal
0
290
Gemini CLIの"強み"を知る! Gemini CLIとClaude Codeを比較してみた!
kotahisafuru
2
210
11年かかって やっとVibe Codingに 時代が追いつきましたね
yimajo
0
210
Featured
See All Featured
BBQ
matthewcrist
89
9.8k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
34
5.9k
Done Done
chrislema
184
16k
Side Projects
sachag
455
43k
ReactJS: Keep Simple. Everything can be a component!
pedronauck
667
120k
Practical Orchestrator
shlominoach
189
11k
The Art of Delivering Value - GDevCon NA Keynote
reverentgeek
15
1.6k
Save Time (by Creating Custom Rails Generators)
garrettdimon
PRO
31
1.3k
Raft: Consensus for Rubyists
vanstee
140
7k
RailsConf 2023
tenderlove
30
1.2k
Dealing with People You Can't Stand - Big Design 2015
cassininazir
367
26k
Let's Do A Bunch of Simple Stuff to Make Websites Faster
chriscoyier
507
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