Code review, do you speak it?

Slides from my Agile By Example 2013 talk.

Michal Ostruszka

October 17, 2013

Tweet

More Decks by Michal Ostruszka

See All by Michal Ostruszka

Building modern web applications. Single Responsibility Principle applied

2

430

Building modern web applications. Single Responsibility Principle applied

4

160

You don't need Rails

3

180

JSON API - standards please

2

270

Backendless frontend development

2

320

Things you probably should know about JavaScript that you may not have been told before

6

690

Other Decks in Programming

See All in Programming

AIフル活用時代だからこそ学んでおきたい働き方の心得

0

130

なぜSQLはAIぽく見えるのか/why does SQL look AI like

0

460

AIによるイベントストーミング図からのコード生成 / AI-powered code generation from Event Storming diagrams

2

1.9k

Architectural Extensions

0

290

今から始めるClaude Code超入門

8

8.7k

生成AIを使ったコードレビューで定性的に品質カバー

1

270

例外処理とどう使い分ける？Result型を使ったエラー設計 #burikaigi

16

6.1k

20260127_試行錯誤の結晶を1冊に。著者が解説先輩データサイエンティストからの指南書 / author's_commentary_ds_instructions_guide

1

960

CSC307 Lecture 07

PRO

0

550

副作用をどこに置くか問題：オブジェクト指向で整理する設計判断ツリー

1

610

AI時代のキャリアプラン「技術の引力」からの脱出と「問い」へのいざない / tech-gravity

21

7.2k

360° Signals in Angular: Signal Forms with SignalStore & Resources @ngLondon 01/2026

PRO

0

130

Featured

See All Featured

SEOcharity - Dark patterns in SEO and UX: How to avoid them and build a more ethical web

0

120

ReactJS: Keep Simple. Everything can be a component!

666

130k

Ten Tips & Tricks for a 🌱 transition

0

68

30

1.3k

We Are The Robots

0

160

Optimising Largest Contentful Paint

37

3.6k

実際に使うSQLの書き方徹底解説 / pgcon21j-tutorial

PRO

196

71k

Agile that works and the tools we love

331

21k

SEO for Brand Visibility & Recognition

0

4.2k

CSS Pre-Processors: Stylus, Less & Sass

359

30k

Color Theory Basics | Prateek | Gurzu

0

200

41

3.8k

Transcript

CODE REVIEW DO YOU SPEAK IT? @MOSTRUSZKA | SOFTWAREMILL
what WHY REVIEW CODE FIND YOUR REVIEW STYLE GOOD CODE
REVIEW IS NOT EASY MAKE IT RIGHT & ENJOY IT TIPS FROM THE FIELD
who's zed SOFTWARE WIZARD BOTH-ENDS DEVELOPER AGILE ENTHUSIAST
why TRUCK FACTOR CODE REUSE CHEAP BUG FIXES ASYNCHRONOUS PAIRING
if it is so good WHY NOT ALL DEVELOPERS DO
IT
why TIME, DEADLINES, SCHEDULES WE USED TO SHIP ANYWAY NO
EXPERIENCE IN REVIEW NO PUSH FROM DEVELOPERS TRIED & DROPPED
checkpoint BLOCKING FLOW NON-BLOCKING
toolbox BARE-HANDS TOOL-ASSISTED
doing code review is difficult FOR BOTH SIDES
dive into CODE AUTHOR'S MIND
as a code author EMOTIONS PERSONAL RELATIONS MORE WORK TO
DO RECEIVING FEEDBACK
now step into REVIEWER'S SHOES
as a reviewer ADDITIONAL WORK TO DO CURRENT MOOD PERSONAL
RELATIONS TOXIC CODE OWNERSHIP GIVING FEEDBACK
we are TERRIBLE AT FEEDBACK STUFF
how to MAKE GOOD REVIEW TO SUCK LESS
ideas REVIEW CODE, NOT AUTHOR PRAISE GOOD STUFF BALANCE FEEDBACK
codebrag LIKES: 86 COMMENTS: 877
play with it IT'S NOT SET IN STONE
experiment 1 MONTH KEEP IT OR DROP IT
thank you ENJOY YOUR CODE REVIEWS