Skip to content

Commit 0ed6f60

Browse files
committed
Update time to merge badge from days to hours
Thanks to improvements over the last few months the average time to merge is no longer mulitple days but now can be measured in hours. This change only updates the badge to display the time to merge in hours rather than days. The underlying data has not been changed as to keep the historical results and graphs consistent. Signed-off-by: Brian Carey <[email protected]>
1 parent 6596df3 commit 0ed6f60

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

pkg/constants/main.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ const (
4949
PlotFileName = "plot.png"
5050
MetricsFileName = "metrics"
5151

52-
TimeToMergeBadgeName = "days to merge"
52+
TimeToMergeBadgeName = "hours to merge"
5353
MergeQueueLengthBadgeName = "avg merge queue length"
5454
RetestsToMergeBadgeName = "retests to merge"
5555
MergedPRsBadgeName = "merged PRs"

pkg/output/output.go

+3
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,9 @@ func (b *Handler) writeBadge(name, filePath string, data types.RunningAverageDat
184184
if name == constants.MergedPRsNoRetestBadgeName {
185185
badgeString = data.SimpleBadgeString()
186186
}
187+
if name == constants.TimeToMergeBadgeName {
188+
badgeString = fmt.Sprintf("%.2f ± std %.2f", (data.Avg * 24), (data.Std * 24))
189+
}
187190
err = badge.Render(name, badgeString, color, f)
188191

189192
return err

0 commit comments

Comments
 (0)