util: runtime deprecate toUSVString()

[anonrig]

Let's runtime deprecate toUSVString()

cc @nodejs/tsc

[marco-ippolito]

We usually add a test to check warning is emitted

[RafaelGSS]

Can you provide some context on why we are deprecating it? It's useful for users that will read the v23 changelog.

[benjamingr]

Don't we also need to update the docs?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.04%. Comparing base (291d90a) to head (962ea51).
Report is 270 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #55018   +/-   ##
=======================================
  Coverage   88.04%   88.04%           
=======================================
  Files         652      652           
  Lines      183764   183767    +3     
  Branches    35862    35862           
=======================================
+ Hits       161787   161798   +11     
+ Misses      15233    15223   -10     
- Partials     6744     6746    +2     

Files with missing lines	Coverage Δ
lib/util.js	96.96% <100.00%> (+0.02%)	⬆️

... and 28 files with indirect coverage changes

[mcollina]

I think this will result in a lot of warning on unmaintained modules. What cost is there in keeping this?

[anonrig]

I think this will result in a lot of warning on unmaintained modules. What cost is there in keeping this?

Cost of keeping