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.

Reply via email to