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

Secure Key Crypto - Tech Paper JET Tech Labs
Secure Key Crypto - Tech Paper JET Tech LabsSecure Key Crypto - Tech Paper JET Tech Labs
Secure Key Crypto - Tech Paper JET Tech Labsamber724300
 
FUNCTIONAL AND NON FUNCTIONAL REQUIREMENT
FUNCTIONAL AND NON FUNCTIONAL REQUIREMENTFUNCTIONAL AND NON FUNCTIONAL REQUIREMENT
FUNCTIONAL AND NON FUNCTIONAL REQUIREMENTSneha Padhiar
 
SOFTWARE ESTIMATION COCOMO AND FP CALCULATION
SOFTWARE ESTIMATION COCOMO AND FP CALCULATIONSOFTWARE ESTIMATION COCOMO AND FP CALCULATION
SOFTWARE ESTIMATION COCOMO AND FP CALCULATIONSneha Padhiar
 
Module-1-(Building Acoustics) Noise Control (Unit-3). pdf
Module-1-(Building Acoustics) Noise Control (Unit-3). pdfModule-1-(Building Acoustics) Noise Control (Unit-3). pdf
Module-1-(Building Acoustics) Noise Control (Unit-3). pdfManish Kumar
 
Curve setting (Basic Mine Surveying)_MI10412MI.pptx
Curve setting (Basic Mine Surveying)_MI10412MI.pptxCurve setting (Basic Mine Surveying)_MI10412MI.pptx
Curve setting (Basic Mine Surveying)_MI10412MI.pptxRomil Mishra
 
Stork Webinar | APM Transformational planning, Tool Selection & Performance T...
Stork Webinar | APM Transformational planning, Tool Selection & Performance T...Stork Webinar | APM Transformational planning, Tool Selection & Performance T...
Stork Webinar | APM Transformational planning, Tool Selection & Performance T...Stork
 
Mine Environment II Lab_MI10448MI__________.pptx
Mine Environment II Lab_MI10448MI__________.pptxMine Environment II Lab_MI10448MI__________.pptx
Mine Environment II Lab_MI10448MI__________.pptxRomil Mishra
 
Novel 3D-Printed Soft Linear and Bending Actuators
Novel 3D-Printed Soft Linear and Bending ActuatorsNovel 3D-Printed Soft Linear and Bending Actuators
Novel 3D-Printed Soft Linear and Bending ActuatorsResearcher Researcher
 
Levelling - Rise and fall - Height of instrument method
Levelling - Rise and fall - Height of instrument methodLevelling - Rise and fall - Height of instrument method
Levelling - Rise and fall - Height of instrument methodManicka Mamallan Andavar
 
Gravity concentration_MI20612MI_________
Gravity concentration_MI20612MI_________Gravity concentration_MI20612MI_________
Gravity concentration_MI20612MI_________Romil Mishra
 
KCD Costa Rica 2024 - Nephio para parvulitos
KCD Costa Rica 2024 - Nephio para parvulitosKCD Costa Rica 2024 - Nephio para parvulitos
KCD Costa Rica 2024 - Nephio para parvulitosVictor Morales
 
multiple access in wireless communication
multiple access in wireless communicationmultiple access in wireless communication
multiple access in wireless communicationpanditadesh123
 
STATE TRANSITION DIAGRAM in psoc subject
STATE TRANSITION DIAGRAM in psoc subjectSTATE TRANSITION DIAGRAM in psoc subject
STATE TRANSITION DIAGRAM in psoc subjectGayathriM270621
 
Prach: A Feature-Rich Platform Empowering the Autism Community
Prach: A Feature-Rich Platform Empowering the Autism CommunityPrach: A Feature-Rich Platform Empowering the Autism Community
Prach: A Feature-Rich Platform Empowering the Autism Communityprachaibot
 
Katarzyna Lipka-Sidor - BIM School Course
Katarzyna Lipka-Sidor - BIM School CourseKatarzyna Lipka-Sidor - BIM School Course
Katarzyna Lipka-Sidor - BIM School Coursebim.edu.pl
 
『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书
『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书
『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书rnrncn29
 
Energy Awareness training ppt for manufacturing process.pptx
Energy Awareness training ppt for manufacturing process.pptxEnergy Awareness training ppt for manufacturing process.pptx
Energy Awareness training ppt for manufacturing process.pptxsiddharthjain2303
 
priority interrupt computer organization
priority interrupt computer organizationpriority interrupt computer organization
priority interrupt computer organizationchnrketan
 
Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...
Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...
Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...Erbil Polytechnic University
 
Computer Graphics Introduction, Open GL, Line and Circle drawing algorithm
Computer Graphics Introduction, Open GL, Line and Circle drawing algorithmComputer Graphics Introduction, Open GL, Line and Circle drawing algorithm
Computer Graphics Introduction, Open GL, Line and Circle drawing algorithmDeepika Walanjkar
 

Recently uploaded (20)

Secure Key Crypto - Tech Paper JET Tech Labs
Secure Key Crypto - Tech Paper JET Tech LabsSecure Key Crypto - Tech Paper JET Tech Labs
Secure Key Crypto - Tech Paper JET Tech Labs
 
FUNCTIONAL AND NON FUNCTIONAL REQUIREMENT
FUNCTIONAL AND NON FUNCTIONAL REQUIREMENTFUNCTIONAL AND NON FUNCTIONAL REQUIREMENT
FUNCTIONAL AND NON FUNCTIONAL REQUIREMENT
 
SOFTWARE ESTIMATION COCOMO AND FP CALCULATION
SOFTWARE ESTIMATION COCOMO AND FP CALCULATIONSOFTWARE ESTIMATION COCOMO AND FP CALCULATION
SOFTWARE ESTIMATION COCOMO AND FP CALCULATION
 
Module-1-(Building Acoustics) Noise Control (Unit-3). pdf
Module-1-(Building Acoustics) Noise Control (Unit-3). pdfModule-1-(Building Acoustics) Noise Control (Unit-3). pdf
Module-1-(Building Acoustics) Noise Control (Unit-3). pdf
 
Curve setting (Basic Mine Surveying)_MI10412MI.pptx
Curve setting (Basic Mine Surveying)_MI10412MI.pptxCurve setting (Basic Mine Surveying)_MI10412MI.pptx
Curve setting (Basic Mine Surveying)_MI10412MI.pptx
 
Stork Webinar | APM Transformational planning, Tool Selection & Performance T...
Stork Webinar | APM Transformational planning, Tool Selection & Performance T...Stork Webinar | APM Transformational planning, Tool Selection & Performance T...
Stork Webinar | APM Transformational planning, Tool Selection & Performance T...
 
Mine Environment II Lab_MI10448MI__________.pptx
Mine Environment II Lab_MI10448MI__________.pptxMine Environment II Lab_MI10448MI__________.pptx
Mine Environment II Lab_MI10448MI__________.pptx
 
Novel 3D-Printed Soft Linear and Bending Actuators
Novel 3D-Printed Soft Linear and Bending ActuatorsNovel 3D-Printed Soft Linear and Bending Actuators
Novel 3D-Printed Soft Linear and Bending Actuators
 
Levelling - Rise and fall - Height of instrument method
Levelling - Rise and fall - Height of instrument methodLevelling - Rise and fall - Height of instrument method
Levelling - Rise and fall - Height of instrument method
 
Gravity concentration_MI20612MI_________
Gravity concentration_MI20612MI_________Gravity concentration_MI20612MI_________
Gravity concentration_MI20612MI_________
 
KCD Costa Rica 2024 - Nephio para parvulitos
KCD Costa Rica 2024 - Nephio para parvulitosKCD Costa Rica 2024 - Nephio para parvulitos
KCD Costa Rica 2024 - Nephio para parvulitos
 
multiple access in wireless communication
multiple access in wireless communicationmultiple access in wireless communication
multiple access in wireless communication
 
STATE TRANSITION DIAGRAM in psoc subject
STATE TRANSITION DIAGRAM in psoc subjectSTATE TRANSITION DIAGRAM in psoc subject
STATE TRANSITION DIAGRAM in psoc subject
 
Prach: A Feature-Rich Platform Empowering the Autism Community
Prach: A Feature-Rich Platform Empowering the Autism CommunityPrach: A Feature-Rich Platform Empowering the Autism Community
Prach: A Feature-Rich Platform Empowering the Autism Community
 
Katarzyna Lipka-Sidor - BIM School Course
Katarzyna Lipka-Sidor - BIM School CourseKatarzyna Lipka-Sidor - BIM School Course
Katarzyna Lipka-Sidor - BIM School Course
 
『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书
『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书
『澳洲文凭』买麦考瑞大学毕业证书成绩单办理澳洲Macquarie文凭学位证书
 
Energy Awareness training ppt for manufacturing process.pptx
Energy Awareness training ppt for manufacturing process.pptxEnergy Awareness training ppt for manufacturing process.pptx
Energy Awareness training ppt for manufacturing process.pptx
 
priority interrupt computer organization
priority interrupt computer organizationpriority interrupt computer organization
priority interrupt computer organization
 
Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...
Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...
Comparative study of High-rise Building Using ETABS,SAP200 and SAFE., SAFE an...
 
Computer Graphics Introduction, Open GL, Line and Circle drawing algorithm
Computer Graphics Introduction, Open GL, Line and Circle drawing algorithmComputer Graphics Introduction, Open GL, Line and Circle drawing algorithm
Computer Graphics Introduction, Open GL, Line and Circle drawing algorithm
 

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