We are thinking of adopting Codestriker as the standard way of doing reviews. However, we would like developers to use the topicid in their CVS commits, as a way of verifying that the review has occurred.
Since the topicid is not particularly visible (it is in the e-mails, but only buried in the URL), I modified the Email.pm file to make it easier to do cut-n-paste of the id. Here are my changes: --- Email.orig.pm 2005-12-06 12:48:04.000000000 -0800 +++ Email.pm 2006-12-19 07:24:06.000000000 -0800 @@ -418,8 +418,9 @@ "", "", "", $query->url(), 0); - my $subject = "[REVIEW] Topic $event_name \"" . $topic->{title} . "\" \n"; + my $subject = "[REVIEW:$topic->{topicid}] Topic $event_name \"" . $topic->{title} . "\" \n"; my $body = + "Topic ID: $topic->{topicid}\n" . "Topic \"$topic->{title}\"\n" . "Author: $topic->{author}\n" . (($topic->{bug_ids} ne "") ? "Bug IDs: $topic->{bug_ids}\n" : "") . ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Codestriker-user mailing list Codestriker-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/codestriker-user