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" : "") .
|
