Jazz Forum Welcome to the Jazz Community Forum Connect and collaborate with IBM Engineering experts and users

how to fetch reviewer comments from RTC code review template?

Hello,

We have a use case where in we have to fetch the reviewer comments for a given changeset.

we are trying using OSLC uri.
We have used the below uri to fetch the changeset details and the data fetched is posted below:

https://server_url/ccm/resource/itemOid/com.ibm.team.scm.ChangeSet/_vk7_QSQZEeeesaeXiIPX6w?_mediaType=text/xml

Is there any way to get the reviewer comments on the changeset?
if not OSLC, could you please suggest any other alternatives [like client side SCM API's, Reporting(JRS) etc..]

data fetched:
<scm:ChangeSet itemId="_vk7_QSQZEeeesaeXiIPX6w" properties=""><stateId>_b2PeyCQnEeeesaeXiIPX6w</stateId><immutable>true</immutable><contextId>_Dp6kMdwTEd2jUupDpQV1Rw</contextId><modified>2017-04-18T11:09:04.954Z</modified><mergePredecessor xsi:nil="true"/><predecessor>_UlnJsiQnEeeesaeXiIPX6w</predecessor><active>false</active><comment>Updated the  getAttributeEnumLiteral()</comment><lastUpdatedDate>2017-04-18T11:09:04.829Z</lastUpdatedDate><modifiedBy itemId="_JSPlYIBPEeak7aHXm949Nw"/><changes><internalId>_CeAsISQcEeeesaeXiIPX6w</internalId><kind>2</kind><before>_BMiaBCQYEeeesaeXiIPX6w</before><after>_ht1ZpCQmEeeesaeXiIPX6w</after><item itemId="_fs4eARYOEee0jNErvekYyA" xsi:type="filesystem:FileItemHandle"/></changes><component itemId="__90K0bzgEeSNjv-YQiVrxg"/><owner itemId="_JSPlYIBPEeak7aHXm949Nw"/><xComponentLink xsi:nil="true"/></scm:ChangeSet>

Thanks!

0 votes


Accepted answer

Permanent link

I don't know what you mean by "reviewer comments for a given change set".
It sounds like you might mean the change set 'comment', which is actually the author's comment, and not from a reviewer. It you do want the change set comment, then it looks like you have that information in the data fetched. It would be under the "comment" element (and thus it is "Updated the  getAttributeEnumLiteral()" in this case).

If you want 'reviewer' comments it sounds like you might want to check-out the Code Review feature. In particular, the CLI command which lets you generate report based containing various Code Review metrics. Here is a summary of the Code Review tool if you are not familiar with it:

In Rational Team Concert 6.0.1 we have introduced a new Code Review feature. Code review is integrated with RTC work items and Jazz SCM to provide developers with a seamless, low overhead work flow that is simple to use and is a natural extension to their daily activities. There isn’t a separate server installation, additional licensing fees or other costs associated with using RTC code review. Enabling code review for your projects is simple and takes only a few minutes to set up.

Reviewers can annotate specific lines of code, which helps developers quickly see and fix issues. Issues (or findings) can be marked as must-fix and the reviewer can accept or reject the changes. When preconditions are enabled, a code review cannot be approved if must-fix issues have not been resolved to everyone’s satisfaction.

Developers don’t have to worry about the intricacies of issue management because this is handled for them. Each issue has its own discussion thread and events are logged as the review progresses. Whenever an issue is updated, a new issue raised, or new code changes are added to the review, the events are logged so that you have a complete history of the review life-cycle. Because code reviews are iterative and can include code changes during the review cycle, any annotations on lines of code (issue markers) are automatically re-positioned and visual cues are provided to the reviewer so that they can easily see what has changed since they last looked at the code or since they last reviewed issues.

Team collaboration is accelerated through real-time notification of pending reviews, issues, and fixes. This is becoming more and more important as the number of globally distributed development teams increase. In fact, with the RTC SCM development team spread across three different time zones, it is common for them to have code changes that were written in one time zone reviewed by team members in another time zone.

Code review enables auditing for compliance. Issue status at delivery, approval information, the event logs, and other information captured during the review can be used for a variety of metrics and reporting requirements.

Since the introduction of code review in 6.0.1 more features have been added. Reviewers can inspect intermediate before and after file states which is useful in advanced SCM workflows encountered when multiple developers are working on a common code base. We have added the ability to filter issues on the navigator pane on the left, and various usability enhancements have been added.

The ability to extract work items is useful when an issue needs to be addressed but shouldn’t prevent delivery of the current code changes. The extracted work item captures the required work and will have its own code review cycle.

A SCM CLI command is also available to export code review data such as issues raised, issue activity, reviewer information, code delivery status, and other metrics in JSON or XML format. This enables customers to use their existing reporting tools or custom scripts for auditability and other reporting requirements

Additional Information:
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

Michael Valenta selected this answer as the correct answer

0 votes

Your answer

Register or log in to post your answer.

Dashboards and work items are no longer publicly available, so some links may be invalid. We now provide similar information through other means. Learn more here.

Search context
Follow this question

By Email: 

Once you sign in you will be able to subscribe for any updates here.

By RSS:

Answers
Answers and Comments
Question details
× 10,937
× 1,381
× 149
× 62

Question asked: Apr 21 '17, 2:04 a.m.

Question was seen: 3,213 times

Last updated: Oct 06 '17, 3:51 p.m.

Confirmation Cancel Confirm