SlideShare a Scribd company logo
1 of 17
Better reviewer
By Hieu Nguyen
WHY?
How to become
better at code
reviewing?
Good reviewer?
Good review
● Correct
● Insightful
● Professional
● ...
HOW?
● Better understanding
● More knowledge
● More experience
Good reviewer
comes from Good developer
Curiosity
1. Why do they do it?
2. Why do they do it like that?
3. If I was them, how will I do it?
4. How is my approach compared to them?
Attitude
● Look at the code, not the people
● No compromise
● Don’t scare of being wrong
HOW?
Make it a habit
● Choose a time
● Choose the PR
● Read the code and review it
● Repeat. Every. Single. Day.
● No excuses
Good developer? ● Read. Read. Read a lot
● Write more (code, articles,
books, …)By improving your technical ability
http://aimforsimplicity.com/post/316daysofcode/
Question?
Thank you for listening

More Related Content

What's hot

Year 12 Feedback workshop
Year 12 Feedback workshopYear 12 Feedback workshop
Year 12 Feedback workshopmc0806
 
Raf3 inference and deduction
Raf3 inference and deductionRaf3 inference and deduction
Raf3 inference and deductionHelen Magley
 
7 simple steps to write professionally
7 simple steps to write professionally7 simple steps to write professionally
7 simple steps to write professionallyAL Services
 

What's hot (6)

The Power of Questioning - Prof. R.Lico
The Power of Questioning - Prof. R.LicoThe Power of Questioning - Prof. R.Lico
The Power of Questioning - Prof. R.Lico
 
Year 12 Feedback workshop
Year 12 Feedback workshopYear 12 Feedback workshop
Year 12 Feedback workshop
 
Geek Quotes
Geek QuotesGeek Quotes
Geek Quotes
 
Raf3 inference and deduction
Raf3 inference and deductionRaf3 inference and deduction
Raf3 inference and deduction
 
4. proposal
4. proposal4. proposal
4. proposal
 
7 simple steps to write professionally
7 simple steps to write professionally7 simple steps to write professionally
7 simple steps to write professionally
 

Similar to Better reviewer

Coding Words, A Practical Guide for Developers Who Want To Become Better Writers
Coding Words, A Practical Guide for Developers Who Want To Become Better WritersCoding Words, A Practical Guide for Developers Who Want To Become Better Writers
Coding Words, A Practical Guide for Developers Who Want To Become Better WritersJonathan Bossenger
 
Effective Peer Review
Effective Peer ReviewEffective Peer Review
Effective Peer ReviewJoel Hooks
 
How to Crack a GROUP DISCUSSION: DOs and DONTs of GD
How to Crack a GROUP DISCUSSION: DOs and DONTs of GDHow to Crack a GROUP DISCUSSION: DOs and DONTs of GD
How to Crack a GROUP DISCUSSION: DOs and DONTs of GDDr. Aashish Mehra
 
21st century skills. Some ideas about teaching english today and tomorrow.
21st century skills. Some ideas about teaching english today and tomorrow.21st century skills. Some ideas about teaching english today and tomorrow.
21st century skills. Some ideas about teaching english today and tomorrow.The BKC-IH Teacher Training Centre
 
Individual development plan
Individual development planIndividual development plan
Individual development planYaniv Preiss
 
Code review process
Code review processCode review process
Code review processYuki Wajima
 
Desgin Thinking.pptx
Desgin Thinking.pptxDesgin Thinking.pptx
Desgin Thinking.pptxchirag680966
 
Growth Hacking Workshop
Growth Hacking WorkshopGrowth Hacking Workshop
Growth Hacking WorkshopErik Snyder
 
Interactive read aloud
Interactive read aloudInteractive read aloud
Interactive read aloudjbigsby
 
Design Thinking: The Secret To Succeed In E-Commerce
Design Thinking: The Secret To Succeed In E-CommerceDesign Thinking: The Secret To Succeed In E-Commerce
Design Thinking: The Secret To Succeed In E-CommerceKali Charan Rajput (KC)
 
Group Discussion-DEEKSHA.pptx
Group Discussion-DEEKSHA.pptxGroup Discussion-DEEKSHA.pptx
Group Discussion-DEEKSHA.pptxDeeksha Kadam
 
Importance Of Essay Writing​​
Importance Of Essay Writing​​Importance Of Essay Writing​​
Importance Of Essay Writing​​Online Class Expert
 
Audience Research on a Dime - NTC
Audience Research on a Dime - NTCAudience Research on a Dime - NTC
Audience Research on a Dime - NTCCourtney Clark
 
Group Discussion
Group Discussion Group Discussion
Group Discussion AtifRaza46
 
Job Readiness Masterclass
Job Readiness MasterclassJob Readiness Masterclass
Job Readiness MasterclassJessica Akano
 
Express your designer self through writing
Express your designer self through writingExpress your designer self through writing
Express your designer self through writingYingying Zhang
 
Interview Nuances - Part I
Interview Nuances - Part IInterview Nuances - Part I
Interview Nuances - Part IPravalika75
 
How to review a pull request
How to review a pull requestHow to review a pull request
How to review a pull requestrouanw
 
Activate Agile 2014 : roles, activities, behaviours in Agile Projects
Activate Agile 2014 : roles, activities, behaviours in Agile ProjectsActivate Agile 2014 : roles, activities, behaviours in Agile Projects
Activate Agile 2014 : roles, activities, behaviours in Agile Projectsdeancornish
 

Similar to Better reviewer (20)

Coding Words, A Practical Guide for Developers Who Want To Become Better Writers
Coding Words, A Practical Guide for Developers Who Want To Become Better WritersCoding Words, A Practical Guide for Developers Who Want To Become Better Writers
Coding Words, A Practical Guide for Developers Who Want To Become Better Writers
 
Effective Peer Review
Effective Peer ReviewEffective Peer Review
Effective Peer Review
 
How to Crack a GROUP DISCUSSION: DOs and DONTs of GD
How to Crack a GROUP DISCUSSION: DOs and DONTs of GDHow to Crack a GROUP DISCUSSION: DOs and DONTs of GD
How to Crack a GROUP DISCUSSION: DOs and DONTs of GD
 
21st century skills. Some ideas about teaching english today and tomorrow.
21st century skills. Some ideas about teaching english today and tomorrow.21st century skills. Some ideas about teaching english today and tomorrow.
21st century skills. Some ideas about teaching english today and tomorrow.
 
Individual development plan
Individual development planIndividual development plan
Individual development plan
 
Code review process
Code review processCode review process
Code review process
 
Desgin Thinking.pptx
Desgin Thinking.pptxDesgin Thinking.pptx
Desgin Thinking.pptx
 
Growth Hacking Workshop
Growth Hacking WorkshopGrowth Hacking Workshop
Growth Hacking Workshop
 
Interactive read aloud
Interactive read aloudInteractive read aloud
Interactive read aloud
 
Design Thinking: The Secret To Succeed In E-Commerce
Design Thinking: The Secret To Succeed In E-CommerceDesign Thinking: The Secret To Succeed In E-Commerce
Design Thinking: The Secret To Succeed In E-Commerce
 
Groupdiscussion
GroupdiscussionGroupdiscussion
Groupdiscussion
 
Group Discussion-DEEKSHA.pptx
Group Discussion-DEEKSHA.pptxGroup Discussion-DEEKSHA.pptx
Group Discussion-DEEKSHA.pptx
 
Importance Of Essay Writing​​
Importance Of Essay Writing​​Importance Of Essay Writing​​
Importance Of Essay Writing​​
 
Audience Research on a Dime - NTC
Audience Research on a Dime - NTCAudience Research on a Dime - NTC
Audience Research on a Dime - NTC
 
Group Discussion
Group Discussion Group Discussion
Group Discussion
 
Job Readiness Masterclass
Job Readiness MasterclassJob Readiness Masterclass
Job Readiness Masterclass
 
Express your designer self through writing
Express your designer self through writingExpress your designer self through writing
Express your designer self through writing
 
Interview Nuances - Part I
Interview Nuances - Part IInterview Nuances - Part I
Interview Nuances - Part I
 
How to review a pull request
How to review a pull requestHow to review a pull request
How to review a pull request
 
Activate Agile 2014 : roles, activities, behaviours in Agile Projects
Activate Agile 2014 : roles, activities, behaviours in Agile ProjectsActivate Agile 2014 : roles, activities, behaviours in Agile Projects
Activate Agile 2014 : roles, activities, behaviours in Agile Projects
 

Recently uploaded

Microscopic Analysis of Ceramic Materials.pptx
Microscopic Analysis of Ceramic Materials.pptxMicroscopic Analysis of Ceramic Materials.pptx
Microscopic Analysis of Ceramic Materials.pptxpurnimasatapathy1234
 
Study on Air-Water & Water-Water Heat Exchange in a Finned Tube Exchanger
Study on Air-Water & Water-Water Heat Exchange in a Finned Tube ExchangerStudy on Air-Water & Water-Water Heat Exchange in a Finned Tube Exchanger
Study on Air-Water & Water-Water Heat Exchange in a Finned Tube ExchangerAnamika Sarkar
 
HARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICS
HARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICSHARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICS
HARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICSRajkumarAkumalla
 
Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝
Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝
Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝soniya singh
 
(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service
(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service
(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Serviceranjana rawat
 
High Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur Escorts
High Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur EscortsHigh Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur Escorts
High Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur EscortsCall Girls in Nagpur High Profile
 
Introduction to IEEE STANDARDS and its different types.pptx
Introduction to IEEE STANDARDS and its different types.pptxIntroduction to IEEE STANDARDS and its different types.pptx
Introduction to IEEE STANDARDS and its different types.pptxupamatechverse
 
(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...
(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...
(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...ranjana rawat
 
MANUFACTURING PROCESS-II UNIT-2 LATHE MACHINE
MANUFACTURING PROCESS-II UNIT-2 LATHE MACHINEMANUFACTURING PROCESS-II UNIT-2 LATHE MACHINE
MANUFACTURING PROCESS-II UNIT-2 LATHE MACHINESIVASHANKAR N
 
Architect Hassan Khalil Portfolio for 2024
Architect Hassan Khalil Portfolio for 2024Architect Hassan Khalil Portfolio for 2024
Architect Hassan Khalil Portfolio for 2024hassan khalil
 
APPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICS
APPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICSAPPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICS
APPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICSKurinjimalarL3
 
(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts
(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts
(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escortsranjana rawat
 
Software Development Life Cycle By Team Orange (Dept. of Pharmacy)
Software Development Life Cycle By  Team Orange (Dept. of Pharmacy)Software Development Life Cycle By  Team Orange (Dept. of Pharmacy)
Software Development Life Cycle By Team Orange (Dept. of Pharmacy)Suman Mia
 
VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130
VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130
VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130Suhani Kapoor
 
Call Girls Delhi {Jodhpur} 9711199012 high profile service
Call Girls Delhi {Jodhpur} 9711199012 high profile serviceCall Girls Delhi {Jodhpur} 9711199012 high profile service
Call Girls Delhi {Jodhpur} 9711199012 high profile servicerehmti665
 
GDSC ASEB Gen AI study jams presentation
GDSC ASEB Gen AI study jams presentationGDSC ASEB Gen AI study jams presentation
GDSC ASEB Gen AI study jams presentationGDSCAESB
 
Decoding Kotlin - Your guide to solving the mysterious in Kotlin.pptx
Decoding Kotlin - Your guide to solving the mysterious in Kotlin.pptxDecoding Kotlin - Your guide to solving the mysterious in Kotlin.pptx
Decoding Kotlin - Your guide to solving the mysterious in Kotlin.pptxJoão Esperancinha
 
High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...
High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...
High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...Call Girls in Nagpur High Profile
 
Call Girls in Nagpur Suman Call 7001035870 Meet With Nagpur Escorts
Call Girls in Nagpur Suman Call 7001035870 Meet With Nagpur EscortsCall Girls in Nagpur Suman Call 7001035870 Meet With Nagpur Escorts
Call Girls in Nagpur Suman Call 7001035870 Meet With Nagpur EscortsCall Girls in Nagpur High Profile
 

Recently uploaded (20)

Microscopic Analysis of Ceramic Materials.pptx
Microscopic Analysis of Ceramic Materials.pptxMicroscopic Analysis of Ceramic Materials.pptx
Microscopic Analysis of Ceramic Materials.pptx
 
Study on Air-Water & Water-Water Heat Exchange in a Finned Tube Exchanger
Study on Air-Water & Water-Water Heat Exchange in a Finned Tube ExchangerStudy on Air-Water & Water-Water Heat Exchange in a Finned Tube Exchanger
Study on Air-Water & Water-Water Heat Exchange in a Finned Tube Exchanger
 
HARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICS
HARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICSHARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICS
HARDNESS, FRACTURE TOUGHNESS AND STRENGTH OF CERAMICS
 
Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝
Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝
Model Call Girl in Narela Delhi reach out to us at 🔝8264348440🔝
 
(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service
(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service
(RIA) Call Girls Bhosari ( 7001035870 ) HI-Fi Pune Escorts Service
 
High Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur Escorts
High Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur EscortsHigh Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur Escorts
High Profile Call Girls Nagpur Meera Call 7001035870 Meet With Nagpur Escorts
 
Introduction to IEEE STANDARDS and its different types.pptx
Introduction to IEEE STANDARDS and its different types.pptxIntroduction to IEEE STANDARDS and its different types.pptx
Introduction to IEEE STANDARDS and its different types.pptx
 
(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...
(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...
(ANVI) Koregaon Park Call Girls Just Call 7001035870 [ Cash on Delivery ] Pun...
 
MANUFACTURING PROCESS-II UNIT-2 LATHE MACHINE
MANUFACTURING PROCESS-II UNIT-2 LATHE MACHINEMANUFACTURING PROCESS-II UNIT-2 LATHE MACHINE
MANUFACTURING PROCESS-II UNIT-2 LATHE MACHINE
 
Architect Hassan Khalil Portfolio for 2024
Architect Hassan Khalil Portfolio for 2024Architect Hassan Khalil Portfolio for 2024
Architect Hassan Khalil Portfolio for 2024
 
APPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICS
APPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICSAPPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICS
APPLICATIONS-AC/DC DRIVES-OPERATING CHARACTERISTICS
 
(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts
(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts
(MEERA) Dapodi Call Girls Just Call 7001035870 [ Cash on Delivery ] Pune Escorts
 
Software Development Life Cycle By Team Orange (Dept. of Pharmacy)
Software Development Life Cycle By  Team Orange (Dept. of Pharmacy)Software Development Life Cycle By  Team Orange (Dept. of Pharmacy)
Software Development Life Cycle By Team Orange (Dept. of Pharmacy)
 
VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130
VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130
VIP Call Girls Service Hitech City Hyderabad Call +91-8250192130
 
Call Us -/9953056974- Call Girls In Vikaspuri-/- Delhi NCR
Call Us -/9953056974- Call Girls In Vikaspuri-/- Delhi NCRCall Us -/9953056974- Call Girls In Vikaspuri-/- Delhi NCR
Call Us -/9953056974- Call Girls In Vikaspuri-/- Delhi NCR
 
Call Girls Delhi {Jodhpur} 9711199012 high profile service
Call Girls Delhi {Jodhpur} 9711199012 high profile serviceCall Girls Delhi {Jodhpur} 9711199012 high profile service
Call Girls Delhi {Jodhpur} 9711199012 high profile service
 
GDSC ASEB Gen AI study jams presentation
GDSC ASEB Gen AI study jams presentationGDSC ASEB Gen AI study jams presentation
GDSC ASEB Gen AI study jams presentation
 
Decoding Kotlin - Your guide to solving the mysterious in Kotlin.pptx
Decoding Kotlin - Your guide to solving the mysterious in Kotlin.pptxDecoding Kotlin - Your guide to solving the mysterious in Kotlin.pptx
Decoding Kotlin - Your guide to solving the mysterious in Kotlin.pptx
 
High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...
High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...
High Profile Call Girls Nashik Megha 7001305949 Independent Escort Service Na...
 
Call Girls in Nagpur Suman Call 7001035870 Meet With Nagpur Escorts
Call Girls in Nagpur Suman Call 7001035870 Meet With Nagpur EscortsCall Girls in Nagpur Suman Call 7001035870 Meet With Nagpur Escorts
Call Girls in Nagpur Suman Call 7001035870 Meet With Nagpur Escorts
 

Better reviewer

Editor's Notes

  1. I will not talk about why here as most of us know about why should we do it
  2. Because we cannot improve something we don’t know, let’s think about how a reviewer can be called “good” before answering the previous question.
  3. Good reviewer is the one who can provide good review, and how is a review that can be called good
  4. How to have a review that is correct, insightful and professional? Correct comes from knowledge, experience Insightful comes from curiosity Professional comes from attitude
  5. Good reviewer needs to be good developer (even though the opposite is not necessary true). There are no shortcut around that, if you want your review to be good, then you need to strive to be good developer first. Though, don’t be discouraged if you think you’re novice. Read more code and review them is also a way to make your code better.
  6. If technical ability helps you in creating correct review, then curiosity helps you create insightful one. If you cannot answer any questions in these, ask the creator of the pull request.
  7. Given that we know the ability, then how can we achieve them?
  8. The most important thing is continuation. Making it a habit will remove the need of willpower, and making reviewing effortless
  9. This deserves a book, so I will skim through it