89 results found
-
Allow reviews to be closed with project specific number of reviewers accepting
I would like the capability to specify at the project level the minimum number of "Accepts" allowed such that a review can be closed cleanly when that number is either met or exceeded. That way available reviewers can review without needing the entire list to accept.
4 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.
13 votes -
Add Due Date to the code reviews
To have Due Date associated with code review and let code reviewers know that due date is coming up via email will be great functionality to have.
Basically - coder should be able to add a date for code review deadline and reviewer should get ample notification if code review is not completed within that due date.
10 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.
51 votes -
Add Thumbs Up or Like to a comment
It would be nice to have a thumbs up or like type functionality to allow agreements to other comments in a review.
7 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)60 votes -
Ability for all reviewers to change their decision (Accepted/Rejected) any time they want, without need to wait for review to be updated or
I, as reviewer, can accidently set review status "Accepted"
or I can change my decision after having a conversation with review author
at least I can delete my comment-“defect”, so I can change decision from rejected to accepted.3 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.
35 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)?
32 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.
19 votes -
Ability to hide or collapse comments in review panel
Code reviews with a lot of comments become very ugly with the way comments are organized… and no way to collapse.
having the ability to hide reviewed comments, or collapse comments and retain collapsed/hidden state across sessions for current user will help make the UI more user friendly.5 votes -
Changeset Selection Window Improvements
Would like the ability to auto filter the changeset selection window by the currently logged in user or user entered value (that is kept from session to session.)
The filter when adding changesets for code review is a little convoluted, the UI makes it seems like you need to enter in the full AD account name.
2 votes -
Add exclusion list for filetypes that do not need to be reviewed
Some filetypes do not contain handwritten sourcecode, so they do not need to be reviewed. These file clutter the list of files to be reviewed. E.g. .csproj, .vspscc, .sln, .svcinfo, .disco, .wsdl
Please allow an exclusion list for filetypes that do not need to be reviewed. So these files do not appear anymore in the list.
7 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.
47 votes -
Option for changing the font size
Add an option for changing the font size. The comments are hard to read because of the small font size.
2 votes -
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.
Thanks84 votes -
Show more information in the list of reviews
The review overview list is very minimal. It would be nice to get more information at one glance. Now you have to open each review separatly or hover over the reviews to see things like:
- What tasknumbers and/or changesets are involved in a review.
- How many reviewers have accepted already1 vote -
Search revisions by comment feature
It is important to be able to select commits for review by searching comments field. It is hard to select commits for review based only on author name and branch name.
5 votes -
Allow enter key to switch from diff view to working copy view
The GitHub Visual Studio extension v2.4.3.1737 just did this and it's pure genius. See https://haacked.com/archive/2018/03/15/github-vs-pr-features/ for details. This would probably be easier to add this feature than setting up the UI and persistence to implement the idea at https://devart.uservoice.com/forums/145340-review-assistant/suggestions/5744139-option-to-set-a-default-action-when-opening-compar and nets the same result.
4 votes -
Add a possiblity to specify computer name in email links.
Computer name visible on network can differ from the name detected by RA server. So links in notifications cannot be opened. it would be great if there will be a possibility to override this name in RA server config.
1 vote
- Don't see your idea?