Skip to content

Commit 489ea80

Browse files
committed
fix: unmerged timeToMerge
1 parent c1953df commit 489ea80

1 file changed

Lines changed: 11 additions & 7 deletions

File tree

apps/api/src/app/pull-requests/resolvers/transformers/pull-request-tracking.transformer.ts

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,12 @@ export const transformPullRequestTracking = ({
5353
firstReviewAt: tracking.firstReviewAt?.toISOString(),
5454
timeToCode: tracking.timeToCode,
5555
firstDeployedAt: tracking.firstDeployedAt?.toISOString(),
56+
timeToFirstReview: calculateTimeForEvent({
57+
from: tracking.firstReadyAt || pullRequest.createdAt,
58+
duration: tracking.timeToFirstReview,
59+
pullRequest,
60+
uselessAfterMerge: true,
61+
}),
5662
timeToFirstApproval: calculateTimeForEvent({
5763
from:
5864
tracking.firstReviewAt ||
@@ -62,14 +68,12 @@ export const transformPullRequestTracking = ({
6268
pullRequest,
6369
uselessAfterMerge: true,
6470
}),
65-
timeToFirstReview: calculateTimeForEvent({
66-
from: tracking.firstReadyAt || pullRequest.createdAt,
67-
duration: tracking.timeToFirstReview,
68-
pullRequest,
69-
uselessAfterMerge: true,
70-
}),
7171
timeToMerge: calculateTimeForEvent({
72-
from: tracking.firstReadyAt || pullRequest.createdAt,
72+
from:
73+
tracking.firstApprovalAt ||
74+
tracking.firstReviewAt ||
75+
tracking.firstReadyAt ||
76+
pullRequest.createdAt,
7377
duration: tracking.timeToMerge,
7478
pullRequest,
7579
uselessAfterMerge: true,

0 commit comments

Comments
 (0)