Skip to content
GitLab
Next
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • GitLab GitLab
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 44,763
    • Issues 44,763
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Merge requests 1,329
    • Merge requests 1,329
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GitLab.orgGitLab.org
  • GitLabGitLab
  • Issues
  • #38062
You need to sign in or sign up before continuing.
Closed
Open
Issue created Dec 04, 2019 by Jeremy Watson (ex-GitLab)@jeremy-glContributor

Code Review Analytics MVC

Overview

As part of #38014 (closed), we're exploring adding analytics into the application that are dedicated to better understanding patterns and trends around code review. The effort represented by that issue involves discovery and design, but does not deliver additions to GitLab.

This issue serves as the Build issue to implement the designed solution in #38014 (closed).

Proposed Approach

Visual representation

Code_Review_Analytics_MVC_v4-removed-percentile-and-fix-definition

Terms and Details

  • Merge Requests in review are all open MRs for given project
  • Review time is time from first non-author comment to merge (or now) in days/hours (adjust notice text accordingly)
  • Review comment is a comment on a MR from a human and non-author.
  • Approvers are people who already approved a MR
  • Percentile (for a future iteration) represents number of MRs with review time less or equal to given MR review time normalized to percents. E.g. 93% percentile means that a MR review time is more than or equal to 93% of MRs in review. Lower number - better review time.
  • In the project analytics sidebar, introduce Code Review as a new project-level analytics feature
  • Search filter bar should support filter by milestone and labels.
  • Code Review analytics should be available in GitLab Starter+ license or Bronze+ plan.
  • Paginated in pages of 10 MRs.
  • Sort by review time in descending order.
  • Only accessible by the Reporter role and higher.
Edited Jan 09, 2020 by Dan Jensen
Assignee
Assign to
Time tracking