Review Assistant
Review Assistant is a code review plugin for Visual Studio. Peer code review tool helps you to create review requests and respond to them without leaving Visual Studio, improve code quality and ship software with less defects, automate the process with minimum overhead.
http://www.devart.com/review-assistant/
-
Review Board in the browser
Add an lightweight Review Board displayed as a web page at server.
As a development manager I need to track many projects and I don't want to run Visual Studio for every of them just to watch is there any update.
Or any other way of notification/summary information distributed by other means than Visual Studio.
Thanks78 votes -
Support pre-commit review for all VCS
Pre-commit review is must to have for all type of version controls with diff/patch files or directly by uploading different version of files otherwise its going to be behind the other review tools in making of choice.
41 votes -
JIRA Integration
Provide the following integration points with JIRA:
- Associate code reviews with JIRA issues (e.g. review for a feature)
- Turn comments into issues (defects)40 votes -
Comparison highlighting only changes made by revisions
If someone submits a review that includes several revisions spanning a period of time, there's no way to determine what changes were made by the author. The only options (Compare Base and Last Revisions, etc) deal with the Base and Last Revisions. Effectively, this includes any changes made by *anyone* in-between, not just the author.
36 votes -
API for all basic interactions
There are over 60 requests for some API level access to the product. There are a ton of standard and custom issue tracking systems out there that could tie into your product. In our case we would love to tie TFS check in events to create and update reviews automatically. This seems like a major missing feature compared to a lot of other products in the space. Yours would work best if the API was exposed. People have been asking for over 2 years.
The command line is clunky and insuffiecient, or at least not documented well enough to be…
31 votes -
Add link back to Work Item in TFS after code review complete/rejected
It would be good if the code review could log back to the original work item that a code review has taken place. When doing a review, it shows me the Work Items attached to the changeset (this is nice). However, when the review is done (accepted or rejected), there is no log in the Work Item that the review took place.
This has currently got to be done manually, would be great to get this full link.
Is there any way to link it or log something in the history of the work item(s)?
28 votes -
Ability to categorise comments
I'd love the ability to categorise a comment. Example categories might be coding standard, implementation, functionality. You could then report on these. E.g. Developer A received 20 comments last week, 90% of them were about coding standards. It will help to identify patterns in developers weak and strong points.
27 votes -
Third status for review - "Response Required"
At the moment you can only "accept" or "reject", I'd like to see a third status that doesn't close the review. Thus if reviewer has questions and wants to discuss an issue via comments before deciding on accepting or rejecting they select this status.
The status could be called "Response required"18 votes -
Preview the Diff page when single clicking on a file in the changes list.
When viewing files from the "Changes" list of a code review you have to double click the file-name to have the file display with the Diff view. This then needs to be manually closed.
To improve usability, and speed up the review process, I would like to see the Diff view open in a preview tab in Visual Studio (as when you single click a file in the solution tree) and then automatically close when a new file is clicked on. As is the default behaviour of the preview tab.
Thank you.
17 votes -
View the branch name during code review
Currently its impossible to see the branch name while doing a code review. This makes it hard for the reviewer to switch to the branch to directly make changes to it.
16 votes -
Control client upgrade notices from the server configuration
As a server admin I should be able to control whether my clients get upgrade notices, esp. for breaking changes where both client and server need to be upgraded together. At my company dozens of people have asked why they even get prompted for the 3.x version when the server is still on 2.x. Many companies have change control policies or software packaging regimes which prevent quick software upgrades. The ability to specify at the server that you want to suppress upgrade notices would make the job of administrator easier.
15 votes -
Show links to TFS changeset details included in the review
I'd like to know what exact changesets developer included in the review and want go to changeset detail page of visual studio (just like when browsing history of file and right clicking and choosing changeset details).
There are related work items but not changesets itself.
14 votes -
Accept and Reject Files as a Whole
I really need the ability to accept or reject files as a whole.
Consider reviews with lots of files, or reviews with complex or very interrelated code spread among multiple files. The reviewer is likely to jump back and forth between multiple files to ensure the solution that is being reviews is solid. Once a reviewer has determined that the file is acceptable, they need to be able to mark it as such so that as they continue on they can keep track of that. Otherwise, if they get interrupted during the review (again consider large reviews that likely will…
13 votes -
Ability to search, filter and order comments
Provide more possibilities to search, filter and order the comments.
e.g. for the reviewer's name, file name, ...11 votes -
TFS Shelvesets in Code Coverage report
Currently the Code Coverage report shows only TFS Changeset revisions and their status. It would be helpful if we can see even Shelveset revisions and their status. Teams who rely on pre-commit reviews use only Shelvesets and hence it's not of any use if they can see only Changesets in the Code Coverage report.
11 votes -
Option to set a default action when opening comparison
By default, Review Assistant launches the 'Compare Base and Last Revisions' command when you perform a comparison. Add the option to choose a default comparison action. E.g. during comparison, authors usually use 'Compare Base Revision with Working Copy' to be able to add changes to the working copy file.
11 votes -
Slack Support
As Slack is getting more and more popular, integration with ReviewAssistant would be really useful.
All email notifications could be sent to Slack too.
10 votes -
Renamed/moved files support
Currently, when a file is renamed and/or moved, 2 entries will appear in the changed files list. The old file will appear as deleted and the new file will appear as added. While by itself is only a nuisance, if any changes occur in the added file there is no way to compare them with the deleted file. Since rename/moving a file may also introduce changes in the file itself (like for example a changed namespace and class name) it becomes problematic to code review these cases.
As far as Git is concerned (with the proper git command) there exists…9 votes -
Allow custom reports at the user or global level
It’s becoming really painful to run most of my reports because of the lack of ability to save custom values like filters, project/report type combos and date ranges so I can run them again later. It would be nice if I could save these as a "custom" report entry on the list so I could reuse my filters since the GUI for filters is very combersome for more than one or two custom values. If there could be user-specific or global saved report definitions that would be great, because I would like to allow multiple project owners to be able…
8 votes -
Gravatar integration
Add integration with avatar service - gravatar. It will make easy user identification for Code Review participants. This is implemented in Crucible product for example.
8 votes
- Don't see your idea?