Changes between Version 21 and Version 22 of PeerReviewPlugin


Ignore:
Timestamp:
May 29, 2006, 6:31:25 PM (18 years ago)
Author:
szm
Comment:

Comment added

Legend:

Unmodified
Added
Removed
Modified
  • PeerReviewPlugin

    v21 v22  
    5353We use Trac and Subversion, but when we do a code review, the author of the code changes sends the source files he changed to other engineers via email, and they place these files into their working directory, and use subversion diff to compare the working directory with the latest source in repository. All of the changes made by the author are then very clear, and comments can be sent back to the author via email, incorporated/discussed, and then new code changes sent out for review. Obviously, this approach works, but would be much better if it was integrated into Trac. Anyway, I have looked at your documentation, and unless I am mistaken you require the code changes to be committed into repository first, then your code is used to mark each line that was changed, and needs to be reviewed. Is this correct ? If so, do you have any plans to store the code changes into a temporary location, and use subvresion diff to compare with the HEAD revision in the repository, so that the changes are very clear to the person reviewing the code ?
    5454
     55==== Comment by szm on Ma 30 mai 2006 04:31:24 EST ====
     56This plugin would be much more interesting if it could compare svn diffs rather than having the user mark the changes manually, which is error prone, and difficult to compare against the previous version.
     57
    5558[[AddComment]]