I suggest you ...

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.

170 votes
Vote 0 votes Vote Vote
Vote
Sign in
Check!
(thinking…)
Reset
or sign in with
  • facebook
  • google
    Password icon
    I agree to the terms of service

    You'll receive a confirmation email with a link to create a password (optional).

    Signed in as (Sign out)
    You have left! (?) (thinking…)
    anonymousanonymous shared this idea  ·   ·  Flag idea as inappropriate…  ·  Admin →

    22 comments

    Sign in
    Check!
    (thinking…)
    Reset
    or sign in with
    • facebook
    • google
      Password icon
      I agree to the terms of service

      You'll receive a confirmation email with a link to create a password (optional).

      Signed in as (Sign out)
      Submitting...
      • B SpencerB Spencer commented  ·   ·  Flag as inappropriate

        It would be good to be able to link to an external checklist OR be able to create one dynamically (both requested).

      • Ben PhilipBen Philip commented  ·   ·  Flag as inappropriate

        Checklists may contain procedural items that cannot be scripted such as those associated with ISO or TL compliance.

      • Pawel PieczulPawel Pieczul commented  ·   ·  Flag as inappropriate

        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 KondzielaOla Kondziela commented  ·   ·  Flag as inappropriate

        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 MSAnil MS commented  ·   ·  Flag as inappropriate

        This is very useful feature and this would be prerequisite to start a code review itself !

      • Anil MSAnil MS commented  ·   ·  Flag as inappropriate

        This is very useful feature and this would be prerequisite to start a code review itself !

      • SrikanthSrikanth commented  ·   ·  Flag as inappropriate

        I am not sure how different this extra feature would be when compared to uploading a document while creating the review.

      • Binoy KennyBinoy Kenny commented  ·   ·  Flag as inappropriate

        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..

      • Nikhil WagleNikhil Wagle commented  ·   ·  Flag as inappropriate

        Excellent code review tool. Code review is essential for quality assurance and it is possible to integrate it with CDETS.

      • Gnanendra SatishGnanendra Satish commented  ·   ·  Flag as inappropriate

        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.

      • ManuManu commented  ·   ·  Flag as inappropriate

        This would be useful for better quality of code with less hassles to incorporate the comments

      • AyyamAyyam commented  ·   ·  Flag as inappropriate

        Its really an important tool to have better and quality review of our codes

      ← Previous 1

      Knowledge Base and Helpdesk