code review

Evaluating Developer’s Performance in Code Review Process

Summary: This article describes how to estimate the coverage with code review of code written by individual developers.

The article builds upon Estimating Coverage of Project’s Source Code with Code Review. As from version 2.6, Review Assistant, Devart’s code review tool, provides the new Code Coverage report.

(more…)

Estimating Coverage of Project’s Source Code with Code Review

Summary: This article describes how to estimate the project’s source code coverage with code review. The article also gives an outline of how to make the most from Review Assistant’s Code Coverage report.

(more…)

WPF: Smooth Scrolling With UI Virtualization

During the development of Review Assistant, we encountered a significant performance problem with displaying 100+ comments in one list, as creation of each comment takes a considerable amount of time. At first sight, it seemed that enabling virtualization would fix the problem, but we faced the following obstacles:

  1. When scrolling, the elements jump form one to another without any smoothness. In addition, the thumb height begins to change in size, what looks just weird.
  2. When selecting IsPixelBased in true, the elements are scrolled smoothly, but a lot of them begin to lag. And the worst thing is that sometimes the application crashes with StackOverflowException. The crash is caused by the code in the VirtualizingStackPanel.MeasureOverrideImpl method, where the tail call is used, and the call depth is not limited by any means. (more…)

Creating TFS Custom Check-in Policy – Part 2

This is the second article of the two-part series that explains implementation of the TFS custom check-in policy for pre-commit code review. We developed this policy for Review Assistant – our code review tool.

In the first article we explained what is a check-in policy and what is the implementation procedure. This article shows nuts and bolts behind our implementation of a check-in policy.

(more…)

Creating TFS Custom Check-in Policy – Part 1

This is the first article of the two-part series that explains implementation of the TFS custom check-in policy for pre-commit code review. We developed this policy for Review Assistant – our code review tool.

(more…)

Agile Code Review Process with Review Assistant

Summary: This article describes a scenario of Review Assistant usage in agile development process. The peculiarity of this scenario is that every team member is allowed to join a code review.

(more…)

Tweak for ‘Simple Review Workflow’

Summary: In Review Assistant 2.0 we have modified the simple review workflow. Now it is possible to add a code-related comment and accept author’s changes, and at the same time leave the review open. (more…)

Iterative Code Review

Summary: This article describes improvements we have introduced into the iterative code review process — the distinctive Review Assistant feature.

(more…)

Reviewing Code from Multiple Repositories in One Review

Summary: This article describes new Review Assistant features, namely: the possibility to work with several repositories within a single review, and also the possibility to add nested repositories into a project.

(more…)

Improved Code Review Comments in Review Assistant

Summary: One of our main missions, while working on the new version of Review Assistant was to improve user experience in general. This article describes how we have solved some UX problems related to the commenting on code.

This is the second article in a series of What’s New in Review Assistant 2.0. First of all  I would recommend reading the previous article “Code Review Board Re-designed“.

(more…)