ci: resolve PR number from head repo/branch for fork PRs - #1041
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1041 +/- ##
==========================================
+ Coverage 89.37% 89.46% +0.09%
==========================================
Files 203 203
Lines 18986 19039 +53
Branches 1769 1782 +13
==========================================
+ Hits 16968 17033 +65
+ Misses 2011 1999 -12
Partials 7 7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
You're right, I didn't address the error; I simply removed the warning message about the Node version. I've rewritten the logic, but it doesn't seem to be directly verifiable in this PR. |
avivkeller
left a comment
There was a problem hiding this comment.
I'm also wary to replace something that works, if it ain't broke don't fix it... why do we need our own logic when an external action works fine
|
👍 Yes, no need to introduce additional maintenance burden by rolling our own action, the upstream works w/ Node.js 24. |
MattIPv4
left a comment
There was a problem hiding this comment.
Can we follow the same pattern as in https://github.com/nodejs/nodejs.org/blob/main/.github/workflows/leave-comment.yml, please, so we have consistent implementations?
|
It turns out there's a well-established logic to refer to; thank you for pointing that out. |
|
@btea could I ask if you're using AI for your contributions and/or your comments? |

Description
The "Leave a comment" workflow fails because
thollander/actions-comment-pull-requestruns on the deprecated Node 20 runtime [1], and the upstream PR upgrading it to Node 24 [2] has been open without activity since March 2026.Replace the action with a local composite action built on actions/github-script. The marker format is kept so existing
tagged comments are updated instead of duplicated.
[1] https://github.com/nodejs/doc-kit/actions/runs/31868656146/job/94973630400
[2] thollander/actions-comment-pull-request#449
Validation
Related Issues
Check List
node --run testand all tests passed.node --run format:check&node --run lint.