Ticket #8 (closed task: fixed)

Opened 5 years ago

Last modified 5 years ago

Design/Code Review

Reported by: chitlesh Owned by: chitlesh
Priority: major Milestone: Fedora 12 'Constantine'
Component: FEL Version: devel
Keywords: review Cc:

Description

FEL should be able to provide at least one Design/Code? Review solution.

This solution should reduce the design/code review time and encourage team culture.

Attachments

peerreview.png (135.2 KB) - added by chitlesh 5 years ago.
peer review plugin
trac-peerreview-plugin.spec (1.7 KB) - added by chitlesh 5 years ago.
draft spec file for peer review

Change History

Changed 5 years ago by chitlesh

peer review plugin

comment:1 Changed 5 years ago by chitlesh

trac peerreview plugin seems to fit into our existing trac package-set.

This package-set includes one location to manage, svn/git repos, tickets and now code review.

I've prepared the RPM of this plugin. Though I managed to load the plugin (with the little knowledge I have on servers), I was not able to file a design/code review. Can you anyone try this rpm and give me some feedback ?

RPM : http://chitlesh.fedorapeople.org/RPMS/trac-peerreview-plugin-2.0-1.svn5357.fc10.noarch.rpm SRPM : http://chitlesh.fedorapeople.org/RPMS/trac-peerreview-plugin-2.0-1.svn5357.fc10.src.rpm

comment:2 Changed 5 years ago by chitlesh

This company provides Code Review facilities. You can read its white paper http://smartbear.com/docs/CodeReviewSocialEffects.pdf

Now we have a lot of opensource equivalent around, trac peerreview plugin as example. If we could get this running, we will extend our EDA portfolio as well.

comment:3 Changed 5 years ago by chitlesh

After some investigation, this problem is because of versions incompatibilities. I've tried the branch 0.10 of the peerreview plugin and it works as expected. Thus there is no barrier for its inclusion except packaging.

Changed 5 years ago by chitlesh

draft spec file for peer review

comment:4 Changed 5 years ago by chitlesh

  • Type changed from defect to task

comment:5 Changed 5 years ago by chitlesh

  • Resolution set to fixed
  • Status changed from new to closed

Package approved and in process to build system

Note: See TracTickets for help on using tickets.