Code Review: Veni, ViDI, ViciYuriy Tymchuk, Andrea Mocci and Michele Lanza

REVEAL @ Faculty of Informatics – University of Lugano, Switzerland

Code review of future

Code Review of Future

What do reviewers expect?

What do tools provide?

What do reviewers expect?What do reviewers expect?

What do reviewers expect?

What do reviewers expect?

Find defects

What do reviewers expect?

Find defectsImprove code

What do reviewers expect?

Find defectsImprove codeFind alternative solutions

What do reviewers have problems with?

What do reviewers have problems with?

What do reviewers do when they don’t understand?

What do reviewers do when they don’t understand?

What do reviewers do when they don’t

What do tools provide?

What do tools provide?

CODEFLOW COLLABORATOR

CRUCIBLE PHABRICATOR

GERRIT GITHUB

REVIEW BOARD UPSOURCE

What do tools provide for diff view?

What do tools provide for code commenting?

What do tools provide for automated quality analysis?

What do tools provide for reviewer selection?

What do tools provide for code navigation?

What do tools provide?

What do tools provide?

Visual Design Inspection

Package

Class

Method

Package

Class

Method

Less Violations

Package

Class

Method

More ViolationsLess Violations

SmallLint Violations

SmallLint Violations115 Rules

SmallLint Violations115 Rules7 Groups

Static Analysis Rules

Main View & System Summary

Selected Entity Critics

& Source Code

Filtering

Fixing

Auto-Fixing

Dedicated Fixing UI

Comments on Entities

Code Review

Code ReviewReview (Code Review)

Review (Code Review)

v3: issues / months ≈ issues/day2390 13 6v4: issues / months ≈ issues/day1200 9 4.5ViDI

v3: issues / months ≈ issues/day2390 13 6v4: issues / months ≈ issues/day1200 9 4.5ViDI

immersive visual inspector

augmented with critics

edit system in-place

automated "xes dedicated "xing tools

annotate entities review of the review session

v3: issues / months ≈ issues/day2390 13 6v4: issues / months ≈ issues/day1200 9 4.5

ViDI

immersive visual inspector

augmented with critics

edit system in-place

v3: issues / months ≈ issues/day2390 13 6v4: issues / months ≈ issues/day1200 9 4.5ViDI

immersive visual inspector

augmented with critics

edit system in-place

automated "xes dedicated "xing tools

annotate entities review of the review session

v3: issues / months ≈ issues/day2390 13 6v4: issues / months ≈ issues/day1200 9 4.5ViDI

immersive visual inspector

augmented with critics

edit system in-place

automated "xes dedicated "xing tools

annotate entities review of the review session

self -"xing custom "x tools

self -"xing custom "x tools

gradual

self -"xing custom "x tools

gradual adjustable

self -"xing custom "x tools

gradual adjustable composite

self -"xing custom "x tools

gradual adjustable composite much more

• • •

ViDIYuriy Tymchuk, Andrea Mocci and Michele Lanza

REVEAL @ Faculty of Informatics – University of Lugano, Switzerland

Veni ViciCode Review: , ,

Yuriy Tymchuk, Andrea Mocci and Michele LanzaREVEAL @ Faculty of Informatics – University of Lugano, Switzerland

ViDIVeni ViciCode Review:

Yuriy Tymchuk, Andrea Mocci and Michele LanzaREVEAL @ Faculty of Informatics – University of Lugano, Switzerland

ViDIVeni ViciCode Review: