Thanks. I just enhanced the plugin with options to support continuous integration / deployment and ticket workflows:
* Can automatically change ticket fields based on result of code review (e.g., send back to implementation) * Can specify a command to execute upon submitting a code review (e.g., jenkins job trigger) * Can restrict what changeset (git support only right now) gets deployed based on status of code reviews (approximates gerrit behavior) I updated the wiki with these details: http://trac-hacks.org/wiki/CodeReviewerPlugin#ContinuousIntegrationJenkinsHudsonBittenetc. Cheers. - Rob On May 16, 11:30 am, Olemis Lang <[email protected]> wrote: > On 5/16/12, Ethan Jucovy <[email protected]> wrote: > > > On Wed, May 16, 2012 at 8:58 AM, Rob Guttman <[email protected]> wrote: > > >> Gerrit <http://code.google.com/p/gerrit/> is a great idea for a code > >> review tool but it ended up being a poor fit for our internal (non-open > >> source project) needs. I've used and considered others (e.g., Crucible, > >> Review Board, GitHub), but what I really wanted for my team was something > >> simple, lightweight and better integrated into our existing Trac ticket > >> workflow. So I built this plugin: > > >> http://trac-hacks.org/wiki/CodeReviewerPlugin > > >> Hope others find this useful. Feedback welcome. > > > This looks really great. I've recently been feeling a need for something > > like this for internal projects myself. The CommitTicketReferenceMacro and > > automatic comments on tickets referenced in the reviewed changeset are > > really nice touches > > [...] > > I second that ! > Great ! > > -- > Regards, > > Olemis. > > Blog ES:http://simelo-es.blogspot.com/ > Blog EN:http://simelo-en.blogspot.com/ > > Featured article: -- You received this message because you are subscribed to the Google Groups "Trac Users" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/trac-users?hl=en.
