+ All Categories
Home > Documents > Lecture 08 03 -...

Lecture 08 03 -...

Date post: 24-Sep-2020
Category:
Upload: others
View: 0 times
Download: 0 times
Share this document with a friend
7
CODE REVIEW Creative Software Architectures for Collaborative Projects CS 130 Donald J. Patterson
Transcript
Page 1: Lecture 08 03 - djp3.westmont.edudjp3.westmont.edu/classes/2016_09_CS130/Lectures/Lecture_08_03.… · Phabricator Review Board Rietveld RhodeCode Understand Upsource Maintainer Atlassian

CODE REVIEW

Creative Software Architectures for Collaborative Projects CS 130 Donald J. Patterson

Page 2: Lecture 08 03 - djp3.westmont.edudjp3.westmont.edu/classes/2016_09_CS130/Lectures/Lecture_08_03.… · Phabricator Review Board Rietveld RhodeCode Understand Upsource Maintainer Atlassian

GOAL: HAVE MULTIPLE EYES ON ALL CODE• Identify obvious logic errors

• Make sure that the code meets the requirements of the

branch

• Make sure new automated tests are sufficient for the

new code

• Decide if existing automated tests need to be rewritten

to account for changes in the code

• Verify that the new code conforms to existing style

guidelines

CODE REVIEW

Page 3: Lecture 08 03 - djp3.westmont.edudjp3.westmont.edu/classes/2016_09_CS130/Lectures/Lecture_08_03.… · Phabricator Review Board Rietveld RhodeCode Understand Upsource Maintainer Atlassian

BENEFITS• Multiple developers know what is going on

• It helps everyone do a better job estimating code

difficulty

• Allows developers to take time off (why?)

• Supports learning from each other

• Peer pressure prevents people taking shortcuts

CODE REVIEW

Page 4: Lecture 08 03 - djp3.westmont.edudjp3.westmont.edu/classes/2016_09_CS130/Lectures/Lecture_08_03.… · Phabricator Review Board Rietveld RhodeCode Understand Upsource Maintainer Atlassian

COSTS• Code reviews take time

CODE REVIEW

Page 6: Lecture 08 03 - djp3.westmont.edudjp3.westmont.edu/classes/2016_09_CS130/Lectures/Lecture_08_03.… · Phabricator Review Board Rietveld RhodeCode Understand Upsource Maintainer Atlassian

CODE REVIEW TOOLS

Page 7: Lecture 08 03 - djp3.westmont.edudjp3.westmont.edu/classes/2016_09_CS130/Lectures/Lecture_08_03.… · Phabricator Review Board Rietveld RhodeCode Understand Upsource Maintainer Atlassian

Recommended