util: update util.isArray() to eol

[miguelmarcondesf]

Refs #58013

Update util.isArray to eol

[cjihrig]

It seems like there is still some hesitation about doing this - #58013 (comment).

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.51%. Comparing base (842448b) to head (8640001).
⚠️ Report is 29 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #58027      +/-   ##
==========================================
+ Coverage   88.00%   88.51%   +0.50%     
==========================================
  Files         704      704              
  Lines      208739   208736       -3     
  Branches    40196    40280      +84     
==========================================
+ Hits       183706   184765    +1059     
+ Misses      16992    15976    -1016     
+ Partials     8041     7995      -46     

Files with missing lines	Coverage Δ
lib/util.js	97.96% <ø> (+1.09%)	⬆️

... and 100 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[geeksilva97]

I think build is failing because a link to the util.isArray still exists at https://github.com/nodejs/node/blob/main/doc/api/deprecations.md?plain=1#L998 and https://github.com/nodejs/node/blob/main/doc/api/deprecations.md?plain=1#L4031.

[github-actions]

Failed to start CI

   ⚠  Commits were pushed since the last approving review:
   ⚠  - util: update util.isArray() to eol
   ⚠  - Update version change with pr number
   ⚠  - fix endoflife typo
   ⚠  - Fix typo from version
   ⚠  - doc: update deprecation notice for util.isArray() removal
   ⚠  - doc: remove unused reference to Array.isArray() in util documentation
   ✘  Refusing to run CI on potentially unsafe PR

https://github.com/nodejs/node/actions/runs/14686825918

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/66500/

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/66515/

[marco-ippolito]

CITGM https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/3587/

[jasnell]

I still encounter this API in the wild a fair amount. I'm not all that convinced we should move this one of EOL just yet.

[RafaelGSS]

+1 Moving this EOL to Node.js 26.

[miguelmarcondesf]

+1 Moving this EOL to Node.js 26.

@RafaelGSS Cool, I'll fix the conflicts today

[jasnell]

For 26, ok. I still think this one is going to end up a bit more disruptive but that's what semver-major is for.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/69337/

[RafaelGSS]

Let's land it only when v25 goes out. Due to the upcoming V8 update, I might need to sync main with v25.x.

[AugustinMauroy]

Bump this can be updated and merged ? FYI userland migrations had furnished a codemod for this dep

[avivkeller]

@miguelmarcondesf rebase, please :-)

[miguelmarcondesf]

@miguelmarcondesf rebase, please :-)

@avivkeller done :-)

[avivkeller]

Can someone run a CITGM?