Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?

Re: Re: Formal code review specifications and reporting format

by Nuke (Scribe)
on May 16, 2002 at 20:24 UTC ( #167103=note: print w/replies, xml ) Need Help??

in reply to Re: Formal code review specifications and reporting format
in thread Formal code review specifications and reporting format

  Those are all good and very valid points. What I'm proposing here is that we create a generic template that could be used both as a checklist and a submission format when the review is complete. As I mentioned, the template would be a starting point, and could/should be modified as needed. Those who are new to code reviews would have a great checklist to get them started, and those who are experienced would have a common format to start with that could be modified to suit the code being reviewed.

  A common submission format would also allow the creation of parsing routines to do all manner of interesting things with the contents of the review.

  As long as folks understand that this is just a starting point, and they keep to the format of the review, I think we could end up with a very flexible and robust system for accomplishing this. I may disagree with you on what needs to be reviewed, but it's moot if we can both review code in whatever way we see fit.

  If we can get an agreement to this, perhaps we could move on to the next stage and talk about the implimentation of the above. (Do we create scripts to help us do this, or just a text template? Maybe a script that spits out a text template based on initial input from the reviewer? :) )


  • Comment on Re: Re: Formal code review specifications and reporting format

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://167103]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others surveying the Monastery: (4)
As of 2018-05-27 06:04 GMT
Find Nodes?
    Voting Booth?