From c9d32d2c231a7f13083f6b0096f284f70415b36d Mon Sep 17 00:00:00 2001 From: Josh Soref <2119212+jsoref@users.noreply.github.com> Date: Fri, 11 Aug 2023 09:08:24 -0400 Subject: [PATCH] Fix logging for items transitioning to stale MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The issue that the stale action finds in this log excerpt *is* stale, it just isn't marked that way yet. Saying that the issue isn't stale is misleading and contradictory. https://github.com/jsoref/stale-bot-debug/actions/runs/5828506506/job/15806335430#step:2:97 ::group::[#2] Issue #2 [#2] Issue #2 [#2] Found this issue last updated at: 2023-08-09T14:33:12Z ... [#2] This issue is not stale ... [#2] This issue should be stale based on the last update date the 09-08-2023 (2023-08-09T14:33:12Z) [#2] This issue should be marked as stale based on the option days-before-issue-stale (​https://github.com/actions/stale#days-before-issue-stale​) (1) [#2] Marking this issue as stale --- dist/index.js | 2 +- src/classes/issues-processor.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dist/index.js b/dist/index.js index 343c8313..441592e2 100644 --- a/dist/index.js +++ b/dist/index.js @@ -603,7 +603,7 @@ class IssuesProcessor { } // Determine if this issue needs to be marked stale first if (!issue.isStale) { - issueLogger.info(`This $$type is not stale`); + issueLogger.info(`This $$type is not marked stale`); const shouldIgnoreUpdates = new ignore_updates_1.IgnoreUpdates(this.options, issue).shouldIgnoreUpdates(); // Should this issue be marked as stale? let shouldBeStale; diff --git a/src/classes/issues-processor.ts b/src/classes/issues-processor.ts index 31bbb99d..71a0d003 100644 --- a/src/classes/issues-processor.ts +++ b/src/classes/issues-processor.ts @@ -446,7 +446,7 @@ export class IssuesProcessor { // Determine if this issue needs to be marked stale first if (!issue.isStale) { - issueLogger.info(`This $$type is not stale`); + issueLogger.info(`This $$type is not marked stale`); const shouldIgnoreUpdates: boolean = new IgnoreUpdates( this.options,