It's all about the answers!

Ask a question

Question about source code review feature in RTC6.0.1M3


Hisayoshi Iikura (112) | asked Sep 14 '15, 8:47 p.m.
retagged Oct 06 '17, 3:57 p.m. by Ken Tessier (84117)

Now I am trying the "source code review" feature of RTC6.0.1M3 according to the article below:
https://jazz.net/wiki/bin/view/Main/ScmCodeReviewUserGuide

I have one question.
The article says in author workflow section that the author visits a work item in the web UI and assign a target stream to the code review summary.
I want to know the reason why the author needs to assign a target stream for the review.

I greatly appreciate the comments from those who knows the specification for this feature.

Accepted answer


permanent link
Evan Hughes (2.4k1018) | answered Sep 24 '15, 5:04 p.m.
JAZZ DEVELOPER
edited Oct 05 '17, 10:45 a.m. by David Lafreniere (4.4k7)

As the technical lead for code review, I'm glad you're trying the feature!

Anyone can set the target stream of the code review. We prefer that the author does it because that shows where the author wants to deliver the change sets. There aren't any limitations on who can set that field because we don't want to block the review if the author sets the field and goes off on vacation (for example).

If the feature gets enough traction, we'd love to include an autodeliver feature as part of code review. In that scenario, when a sufficient number of approvals have been given, the change sets would be delivered to the target stream.

Under the covers we use the target stream to resolve the paths within the change set. It is possible to find those paths in other locations, but an explicit delivery target is most accurate.




For more information about RTC Code Review, see:
Article: Rational Team Concert Code Review
Video: Introduction to Code Review (Part 1)
Video: Configuring Code Review (Part 2)
Video: Performing a Code Review (Part 3)
Knowledge Center: Working With the Code Review Tool

David Lafreniere selected this answer as the correct answer

Your answer


Register or to post your answer.