From 04aceb2e9aafbf4f0ad821d4ff1b945e284b3484 Mon Sep 17 00:00:00 2001 From: Tobias Wilken Date: Sun, 14 Dec 2025 09:45:04 +0100 Subject: [PATCH] fix: correct days-to-merge log output to display actual days The daysToMerge/daysToClose values are stored in seconds (legacy naming), but the log output was displaying raw seconds as if they were days. This made values like -398601 appear as bugs when they actually represented about -4.6 days correctly. Now converts to actual days before logging for clarity. --- src/helpers/pullRequestProcessor.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/helpers/pullRequestProcessor.js b/src/helpers/pullRequestProcessor.js index 04d69c2..4288d95 100644 --- a/src/helpers/pullRequestProcessor.js +++ b/src/helpers/pullRequestProcessor.js @@ -144,11 +144,13 @@ export async function processPullRequests() { prResult.daysToMerge = pullRequestData.times.daysToMerge || null; prResult.daysToClose = pullRequestData.times.daysToClose || null; - const actionDays = + // Note: daysToMerge/daysToClose are actually in seconds (legacy naming) + const actionSeconds = pullRequestData.times.daysToMerge || pullRequestData.times.daysToClose; + const actionDays = actionSeconds / 86400; // Convert to actual days console.log( - `Days to ${pullRequestData.times.action}: ${actionDays}` + `Days to ${pullRequestData.times.action}: ${actionDays.toFixed(2)}` ); // Set GitHub status for this pull request