From 45c42366e40da41c75bac905ba485f595040cd59 Mon Sep 17 00:00:00 2001 From: Brett Blue <84536271+brett0000FF@users.noreply.github.com> Date: Wed, 24 Dec 2025 09:51:45 -0700 Subject: [PATCH 1/3] Fix review state tracking logic State tracking logic breaks when a reviewer adds a comment after an approval --- .github/workflows/codeowner_review_status.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/codeowner_review_status.yml b/.github/workflows/codeowner_review_status.yml index a09f5b21a4e..94bb04fb8e7 100644 --- a/.github/workflows/codeowner_review_status.yml +++ b/.github/workflows/codeowner_review_status.yml @@ -60,9 +60,9 @@ jobs: pull_number: number, }); - // Track latest review state per user - const latestReviews = new Map(); + // Track latest review state per user (ignore commented state) reviews.forEach(review => { + if (review.state === 'COMMENTED') return; const existing = latestReviews.get(review.user.login); if (!existing || review.submitted_at > existing.submitted_at) { latestReviews.set(review.user.login, review); @@ -200,4 +200,4 @@ jobs: description: statusDescription, context: 'Documentation Team Approval' }); - console.log(`Created commit status: ${statusState} - ${statusDescription}`); \ No newline at end of file + console.log(`Created commit status: ${statusState} - ${statusDescription}`); From 3bb988862d1fd247c85ca330eb122e68a3c8ac86 Mon Sep 17 00:00:00 2001 From: Brett Blue Date: Wed, 24 Dec 2025 10:02:57 -0700 Subject: [PATCH 2/3] Test change for workflow fix. --- content/en/opentelemetry/instrument/api_support/nodejs/logs.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/content/en/opentelemetry/instrument/api_support/nodejs/logs.md b/content/en/opentelemetry/instrument/api_support/nodejs/logs.md index d0a76798f25..b3b169aa700 100644 --- a/content/en/opentelemetry/instrument/api_support/nodejs/logs.md +++ b/content/en/opentelemetry/instrument/api_support/nodejs/logs.md @@ -1,5 +1,5 @@ --- -title: Node.js OpenTelemetry Logs API Support +title: Node.js OpenTelemetry Logs API Support (Random Change) code_lang: logs type: multi-code-lang code_lang_weight: 2 From ae338b8d94b6b25d6fdb00994be3b09bf0cfb043 Mon Sep 17 00:00:00 2001 From: Brett Blue <84536271+brett0000FF@users.noreply.github.com> Date: Wed, 24 Dec 2025 10:56:28 -0700 Subject: [PATCH 3/3] Add latestReviews map to track user reviews --- .github/workflows/codeowner_review_status.yml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/.github/workflows/codeowner_review_status.yml b/.github/workflows/codeowner_review_status.yml index 94bb04fb8e7..bd6ec43efa3 100644 --- a/.github/workflows/codeowner_review_status.yml +++ b/.github/workflows/codeowner_review_status.yml @@ -61,6 +61,8 @@ jobs: }); // Track latest review state per user (ignore commented state) + const latestReviews = new Map(); + reviews.forEach(review => { if (review.state === 'COMMENTED') return; const existing = latestReviews.get(review.user.login);