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

Reply via email to