Any plans for a jazz integrated code review plugin?
Hi,
On our project we use Eclipse + RTC (2.0) plug in as our dev environment, with a SCRUM variant for process template. We are just starting to use the review/approvers feature of work items and this leads to the question: are there any plans for a code review tool that integrates with this feature of RTC work items? Such a tool should hook into the review approval functionality on the work items and automatically select the source files referred to by any change sets in the work item, and the reviewers also. Currently, if we used something like Jupiter (or paid for Code Collaborator), we'd have to do these steps manually, and in my opinion the review artefacts produced by these sorts of tools would sit better as attachments on a work item rather than (as at present) being checked in to source control. Are there any other jazz users successfully using any sort of code review tooling? Thanks, Jason |
16 answers
Ralph Schoon (63.1k●3●36●46)
| answered Nov 05 '09, 3:03 p.m.
FORUM ADMINISTRATOR / FORUM MODERATOR / JAZZ DEVELOPER
Hi Jason,
I don't know about code review tools, but since there is an integration with our Rational Software analyzer tool for code quality that is also process enabled I think this is definitely possible. I also think it is a great Idea. We usually hope for partners to do integrations and also provide integration capabilities with the RSDK ond other interfaces like OSLC. You could suggest such a capability in a enhancement request if it is not already there (or support the case if it is). In addition you could suggest your preferred vendor(s) to partner and to create such a cool feature. Just some thoughts, Ralph Hi, |
I don't know of any firm plans, but there's at least one work item open about it:
https://jazz.net/jazz/resource/itemName/com.ibm.team.workitem.WorkItem/82085 e |
Hi, Hi Jason Just to add to Ralph's comments - you can use Rational Software Analyser (RSAR), but in a slightly different way to the one you mention above. It is a different approach, so wanted to expand on it here in case this might work for you. You install RSAR into the shell, and then you get additional process rules available in your process. For example, a developer must code-review using RSAR before they can deliver, etc. This then works like any other process rule, so delivered code must have been checked, and what the change set is, and you can generate the appropriate work item if there is a problem, etc. I like the idea of getting the checks done before you deliver rather than it being a retrospective check. You can also then use RSAR in the build so you can run checks again if you want to but that should not be required unless you have a different set of rules you want to check against. I don't have RTC+RSAR in front of me at the moment to tell you all the rules you can use, but hopefully this makes some sense. regards anthony |
Hi, Interesting that at my office we were having the exact same discussion yesterday! We use Code Collaborator and are very pleased with it. However, it doesn't have hooks yet to submit a review from a RTC changeset. Someone asked about this on one of the blog entries that touts CodeCollaborator's interop with ClearCase -- see http://blog.smartbear.com/the_smartbear_blog/2009/08/ready-for-rational.html. The meat of the answer is "We've looked at Rational Team Concert and are interested in adding support for it. I don't know that it will happen 'soon,' but do stay tuned." So perhaps IBM should prod SmartBear a little to implement that more quickly...? Regards, Mike |
Geoffrey Clemm (30.1k●3●30●35)
| answered Nov 07 '09, 11:08 a.m.
FORUM ADMINISTRATOR / FORUM MODERATOR / JAZZ DEVELOPER
Note that most third party vendors are more interested in what their
potential customers think than what IBM thinks, so I'd encourage any potential SmartBear customers that are interested in an integration between SmartBear and RTC to contact SmartBear directly. Cheers, Geoff micjohnson997 wrote: jason.annadaniwrote: |
I believe "Code review" is very different from "Static analysis" (ie. RSAR).
For collaborative code review, in addition to SmartBear also see http://www.alphaworks.ibm.com/tech/ccrt Freddy jason.annadani wrote: Hi, |
Our initial integration with Rational Team Concert is now available in Code Collaborator v5.0.5028.
This is just a Technology Preview - we're planning to add more features to our integration but we wanted to get the initial set of bits out there so that we could get feedback from folks who have expressed an interest in using Code Collaborator with Rational Team Concert. Documentation on the initial feature set is available here: http://smartbear.com/docs/manual/5.0/rtc_overview.html Thanks, Gregg Sporar Senior Product Manager Smart Bear Software http://blog.smartbear.com |
|
And now we have even more available in the integration between Code Collaborator and Rational Team Concert: http://blog.smartbear.com/the_smartbear_blog/2010/07/rational-team-concert-server-support.html
Gregg Sporar Senior Product Manager Smart Bear Software http://blog.smartbear.com |
I tried integrating the code review plugin with RTC and not successful.
RTC 3.0.1 Windows 7 Professional Edition I followed the below steps: - Downloaded the zipped file from: http://backups.smartbear.s3.amazonaws.com/downloads/ccollab_client_6_5_6503_update_site.zip. - Unzipped to $JAZZ_ROOT/ccollab-update-site. Where JAZZ_ROOT=C:\JazzTeamServer - In the $JAZZ_ROOT/server/conf/ccm/provision_profiles directory, "com.smartbear.ccollab-profile.ini" was copied. Content of "com.smartbear.ccollab-profile.ini": url=file:../../ccollab-update-site featureid=com.smartbear.collaborator.rtc.process.feature - "Request Server Reset" from the Jazz web admin - Restart JTS After all the above steps, I still don't see "Code Collaborator" section in the Advance Properties. Am I missing anything here? Thanks! |
You would definitely need to restart CCM too. You may not need to restart JTS, but it wouldn't hurt. Also, make sure you're checking the CCM admin web UI for the Advanced Properties changes, not JTS's.
|
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.