Feature: PR Size to Time dependency

AlexSim93 · Nov 8, 2024 · 4d76423 · 4d76423
1 parent 9f4951c
commit 4d76423
Show file tree

Hide file tree

Showing 7 changed files with 324 additions and 29 deletions.
diff --git a/build/index.js b/build/index.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "pull-request-analytics-action",
-  "version": "4.1.0",
+  "version": "4.2.0",
   "description": "Generates detailed PR analytics reports within GitHub, focusing on review efficiency and team performance.",
   "main": "build/index.js",
   "scripts": {

diff --git a/src/converters/types.ts b/src/converters/types.ts
@@ -78,6 +78,29 @@ export type Collection = {
   discussions?: Discussion;
   discussionsTypes?: DiscussionType;
   prSizes?: string[];
+  sizes?: Record<
+    string,
+    {
+      timeToApprove: number[];
+      timeToReview: number[];
+      timeToMerge: number[];
+      percentile: {
+        timeToApprove?: number;
+        timeToReview?: number;
+        timeToMerge?: number;
+      };
+      average: {
+        timeToApprove?: number;
+        timeToReview?: number;
+        timeToMerge?: number;
+      };
+      median: {
+        timeToApprove?: number;
+        timeToReview?: number;
+        timeToMerge?: number;
+      };
+    }
+  >;
   reviewsConductedSize?: PullRequestSize[];
   pullRequestsInfo?: PullRequestTimelineInfo[];
 };
diff --git a/src/converters/utils/preparePullRequestStats.ts b/src/converters/utils/preparePullRequestStats.ts
@@ -34,6 +34,30 @@ export const preparePullRequestStats = (collection: Collection) => {
       collection.timeToMerge?.map((el) => el / 60),
       mergeIntervals
     ),
+    sizes: Object.entries(collection?.sizes || {}).reduce(
+      (acc, el) => ({
+        ...acc,
+        [el[0]]: {
+          ...acc[el[0]],
+          percentile: {
+            timeToReview: calcPercentileValue(acc[el[0]]?.timeToReview),
+            timeToApprove: calcPercentileValue(acc[el[0]]?.timeToApprove),
+            timeToMerge: calcPercentileValue(acc[el[0]]?.timeToMerge),
+          },
+          median: {
+            timeToReview: calcMedianValue(acc[el[0]]?.timeToReview),
+            timeToApprove: calcMedianValue(acc[el[0]]?.timeToApprove),
+            timeToMerge: calcMedianValue(acc[el[0]]?.timeToMerge),
+          },
+          average: {
+            timeToReview: calcAverageValue(acc[el[0]]?.timeToReview),
+            timeToApprove: calcAverageValue(acc[el[0]]?.timeToApprove),
+            timeToMerge: calcAverageValue(acc[el[0]]?.timeToMerge),
+          },
+        },
+      }),
+      collection?.sizes || {}
+    ),
     median: {
       timeToReview: calcMedianValue(collection.timeToReview),
       timeToApprove: calcMedianValue(collection.timeToApprove),

diff --git a/src/converters/utils/preparePullRequestTimeline.ts b/src/converters/utils/preparePullRequestTimeline.ts
@@ -1,7 +1,11 @@
 import { getMultipleValuesInput, getValueAsIs } from "../../common/utils";
 import { makeComplexRequest } from "../../requests";
 import { Collection } from "../types";
-import { calcDraftTime, getApproveTime } from "./calculations";
+import {
+  calcDraftTime,
+  getApproveTime,
+  getPullRequestSize,
+} from "./calculations";
 import { calcDifferenceInMinutes } from "./calculations/calcDifferenceInMinutes";
 
 export const preparePullRequestTimeline = (
@@ -89,21 +93,88 @@ export const preparePullRequestTimeline = (
 
   return {
     ...collection,
-    timeToReview: typeof timeToReview === 'number'
-      ? [...(collection?.timeToReview || []), timeToReview]
-      : collection.timeToReview,
-    timeToApprove: typeof timeToApprove === 'number'
-      ? [...(collection?.timeToApprove || []), timeToApprove]
-      : collection.timeToApprove,
-    timeToMerge: typeof timeToMerge === 'number'
-      ? [...(collection?.timeToMerge || []), timeToMerge]
-      : collection.timeToMerge,
-    timeToReviewRequest: typeof timeToReviewRequest === 'number'
-      ? [...(collection?.timeToReviewRequest || []), timeToReviewRequest]
-      : collection.timeToReviewRequest,
-    timeInDraft: typeof timeInDraft === 'number'
-      ? [...(collection?.timeInDraft || []), timeInDraft]
-      : collection.timeInDraft,
+    timeToReview:
+      typeof timeToReview === "number"
+        ? [...(collection?.timeToReview || []), timeToReview]
+        : collection.timeToReview,
+    timeToApprove:
+      typeof timeToApprove === "number"
+        ? [...(collection?.timeToApprove || []), timeToApprove]
+        : collection.timeToApprove,
+    timeToMerge:
+      typeof timeToMerge === "number"
+        ? [...(collection?.timeToMerge || []), timeToMerge]
+        : collection.timeToMerge,
+    timeToReviewRequest:
+      typeof timeToReviewRequest === "number"
+        ? [...(collection?.timeToReviewRequest || []), timeToReviewRequest]
+        : collection.timeToReviewRequest,
+    timeInDraft:
+      typeof timeInDraft === "number"
+        ? [...(collection?.timeInDraft || []), timeInDraft]
+        : collection.timeInDraft,
+    sizes: {
+      ...(collection.sizes || {}),
+      [getPullRequestSize(
+        pullRequestInfo?.additions,
+        pullRequestInfo?.deletions
+      )]: {
+        ...(collection.sizes?.[
+          getPullRequestSize(
+            pullRequestInfo?.additions,
+            pullRequestInfo?.deletions
+          )
+        ] || {}),
+        timeToApprove: timeToApprove
+          ? [
+              ...(collection?.sizes?.[
+                getPullRequestSize(
+                  pullRequestInfo?.additions,
+                  pullRequestInfo?.deletions
+                )
+              ]?.timeToApprove || []),
+              timeToApprove,
+            ]
+          : collection?.sizes?.[
+              getPullRequestSize(
+                pullRequestInfo?.additions,
+                pullRequestInfo?.deletions
+              )
+            ]?.timeToApprove,
+        timeToReview: timeToReview
+          ? [
+              ...(collection?.sizes?.[
+                getPullRequestSize(
+                  pullRequestInfo?.additions,
+                  pullRequestInfo?.deletions
+                )
+              ]?.timeToReview || []),
+              timeToReview,
+            ]
+          : collection?.sizes?.[
+              getPullRequestSize(
+                pullRequestInfo?.additions,
+                pullRequestInfo?.deletions
+              )
+            ]?.timeToReview,
+        timeToMerge: timeToMerge
+          ? [
+              ...(collection?.sizes?.[
+                getPullRequestSize(
+                  pullRequestInfo?.additions,
+                  pullRequestInfo?.deletions
+                )
+              ]?.timeToMerge || []),
+              timeToMerge,
+            ]
+          : collection?.sizes?.[
+              getPullRequestSize(
+                pullRequestInfo?.additions,
+                pullRequestInfo?.deletions
+              )
+            ]?.timeToMerge,
+      },
+    },
     pullRequestsInfo: [
       ...(collection?.pullRequestsInfo || []),
       {