Reviewing Reviews

[Isaac Sherman]

We're trying to determine if this is a good solution and one of the things we need to be able to do is have a non-technically inclined person able to pull up the reviews and generate a report on particular modules, probably at the granularity of a file or directory.  Is that sort of functionality, documenting code reviews that have been performed, built in?  It looks like all the components are there (it seems like an administrator could probably do a query and dump the results to a CSV file), but can it be done by a non-administrator and how much formatting/organizing is already handled by RB out of the box and how much would need to be custom?

Thanks,

[Christian Hammond]

Hi Isaac,

First of all, Happy New Year!

We do offer a paid add-on to Review Board called Power Pack, which includes reporting functionality. Non-technical users can generate a few types of built-in reports, based on any number of users or review groups and a date range:

• The time to first feedback on changes, helping see how long review requests sit around before they're reviewed
• The time to close, helping see how long review requests sit around before they've been completed and are closed
• Review request statistics, showing stats on the numbers of review requests, average issue counts (issues raised during review), and average number that have been dropped (rather than fixed), sorted by user
• Code review statistics, showing the number of code reviewers performed by users, how many Ship Its they've given, average number of issues they find, and average number of those issues that were dropped by the code review author
• Code review relationships, which provide a visual graph that helps quickly see who is reviewing whose changes, which can help determine if some people aren't participating much in the code review process, or who your most-frequent reviewers are

CSV export is also available for these graphs, allowing you to perform your own analytics based on the data going into each graph.

You can see details on those graphs here: https://www.reviewboard.org/docs/powerpack/dev/powerpack/manual/reports/

Power Pack is licensed per-user. There are trial licenses available, which last 30 days, and after the trial they convert into a free 2-person license.

Alternatively, you can always build your own scripts that query the database directly (or the API, but this isn't at all optimal for the kind of data you'll generally want in a report). Another option is to write your own extension to Review Board to query the data you want and export it through some endpoint (like a custom API endpoint), if Power Pack's graphs aren't enough. Extensions are written in Python, and can augment much of Review Board's functionality.

Christian

On Tue, Dec 31, 2019 at 10:03 AM Isaac Sherman <sherm...@gmail.com> wrote:

We're trying to determine if this is a good solution and one of the things we need to be able to do is have a non-technically inclined person able to pull up the reviews and generate a report on particular modules, probably at the granularity of a file or directory.  Is that sort of functionality, documenting code reviews that have been performed, built in?  It looks like all the components are there (it seems like an administrator could probably do a query and dump the results to a CSV file), but can it be done by a non-administrator and how much formatting/organizing is already handled by RB out of the box and how much would need to be custom?

Thanks,

--
Supercharge your Review Board with Power Pack: https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
---
You received this message because you are subscribed to the Google Groups "Review Board Community" group.
To unsubscribe from this group and stop receiving emails from it, send an email to reviewboard...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/reviewboard/cff50348-53f9-4069-8c8a-c78fbe3deac7%40googlegroups.com.

--

Christian Hammond

President/CEO of Beanbag

Makers of Review Board