Add checklist to review process
A checklist to keep track of / remind reviewers ensure the author's code is not only functionally correct but satisfies the companies coding / checkin policy. Also would be a good way to prevent recurring bugs / issues by having a way to remind reviews to check for them.
22 comments
-
B Spencer
commented
It would be good to be able to link to an external checklist OR be able to create one dynamically (both requested).
-
Ben Philip
commented
Checklists may contain procedural items that cannot be scripted such as those associated with ISO or TL compliance.
-
Pawel Pieczul
commented
The tool should actually automatically verify coding standard compliance and show develop places where it is violated. The coding standard should be configurable per location of source file - many different standards may be applicable to a single review (like Linux kernel, X-Server, etc.)
-
Ola Kondziela
commented
In complete agreement with opinions that the feature is crucial to support the code review process! The tool should make it possible to run coding standard checklist by developer before initiating the review and during the formal review as well.
-
Anil MS
commented
This is very useful feature and this would be prerequisite to start a code review itself !
-
Anil MS
commented
This is very useful feature and this would be prerequisite to start a code review itself !
-
Srikanth
commented
I am not sure how different this extra feature would be when compared to uploading a document while creating the review.
-
Binoy Kenny
commented
A good tool for ensuring code quality and process associated with it. Also tracking the comments and verifying them is easy. By having a check list also ensures that the developer have followed the process and tested the issue or feature. In our previous projects we used to follow a checklist for any code review which will capture the issue, the root cause, if tested, complied, followed coding and requirement guidlines etc..
-
Harsha
commented
This seems to be very use full feature to enhance the quality of the product.
-
Nikhil Wagle
commented
Excellent code review tool. Code review is essential for quality assurance and it is possible to integrate it with CDETS.
-
Ulhas Dhuri
commented
Tracking of review comments is easy.
-
Mark Lin
commented
I would love to see this feature.
-
Sudhir sharma
commented
Good and Important without this Code should not be considered as quality code.
-
Senthilkumar Balakrishnan
commented
It is really useful.
-
Gnanendra Satish
commented
Very good feature to be in place for better code quality and the process.
I would prefer a checklist each for both the developer and the reviewer so that common developer centric errors are taken care of before even submitting for review. -
Amit Patel
commented
It is very useful feature for code review
-
Manu
commented
This would be useful for better quality of code with less hassles to incorporate the comments
-
Swami
commented
useful feature to be supported
-
Ayyam
commented
Its really an important tool to have better and quality review of our codes
-
Jeremy Pullen
commented
This would be very useful for us as well.