It is a good idea for GSoC.
-- Best regards, Dennis Schetinin 2013/3/11 Igor Stasenko <[email protected]> > Hello, > > here is some thoughts what i would like to have.. one day. > > imagine guy A did some change(s) to code, and guy B wants to review it. > > Currently we having only one tool which helps a little: MC diff viewer.. > but main tool is still human.. > And it is quite tedious for guy B to go somewhere, copy pieces of code > and/or comment on things.. > > what would be really cool to have is to integrate some steroids into > diff viewer where guy B can press "create a review report" > and write his comments directly under selected change .. then post it > somewhere on web (or send emal), > so guy A can see directly which of his changes made guy B unhappy. > > Going further, and integrating this into pharo development process, as > one guy (stef) said: > automation is good, but no changes should be made into image unless at > least one human took a look at it. > > So, i thinking if we could have more streamlined review process, this > could help a bit (or a lot).. > because personally, for me it is tedious today, because integrator or > reviewer has to do many things manually, > which is not fun and takes too much precious time :) > > I imagining something like opening a window, pressing "get list of > pending reviews from bug tracker" > and then selecting a change, review it and pressing "approve or > disapprove" which then will trigger the rest of automatic > things like integration into image and/or putting entry into bug > tracker with reviewer comments. > > -- > Best regards, > Igor Stasenko. > >
