util: rename CallSite.column to columnNumber

[legendecas]

Align the property names lineNumber and columnNumber.

The name should either be line and column, or lineNumber and columnNumber, but not mixing up. This leads confusing doc like

node/lib/util.js

Lines 337 to 344 in db7a31e

	/**
	* @typedef {object} CallSite // The call site
	* @property {string} scriptName // The name of the resource that contains the
	* script for the function for this StackFrame
	* @property {string} functionName // The name of the function associated with this stack frame
	* @property {number} lineNumber // The number, 1-based, of the line for the associate function call
	* @property {number} columnNumber // The 1-based column offset on the line for the associated function call
	*/

[BridgeAR]

I think we could already add a doc only deprecation in this PR. Otherwise LGTM

[legendecas]

@BridgeAR in the change section, a deprecation note "Property column is deprecated in favor of columnNumber." was added. Are you suggesting adding other form of deprecation notation? AFAICT, frontmatter deprecation (e.g.

node/doc/api/util.md

Line 3130 in db7a31e

deprecated: v6.0.0

) or deprecation section (

node/doc/api/util.md

Line 3133 in db7a31e

> Stability: 0 - Deprecated: Use [`Object.assign()`][] instead.

) do not help in this case as the property its own is as a section.

[BridgeAR]

We just need to assign a new deprecation number and document it in the specific deprecation section: https://github.com/nodejs/node/blob/main/doc/api/deprecations.md

[legendecas]

This API is still "Stability: 1.1 - Active development". Deprecations in experimental APIs do not necesirily need a deprecation code through the deprecation cycle: https://github.com/nodejs/node/blob/main/doc/api/documentation.md#stability-index

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.19%. Comparing base (db7a31e) to head (40a94ec).
Report is 29 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #56584      +/-   ##
==========================================
- Coverage   89.19%   89.19%   -0.01%     
==========================================
  Files         662      662              
  Lines      191794   191797       +3     
  Branches    36927    36919       -8     
==========================================
+ Hits       171076   171078       +2     
+ Misses      13568    13565       -3     
- Partials     7150     7154       +4     

Files with missing lines	Coverage Δ
lib/util.js	97.76% <100.00%> (+<0.01%)	⬆️
src/node_util.cc	86.03% <100.00%> (+0.10%)	⬆️

... and 23 files with indirect coverage changes

[nodejs-github-bot]

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

[nodejs-github-bot]

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

[nodejs-github-bot]

Landed in 1238f0a