doc: fix os.version() Windows API

[cjihrig]

pRtlGetVersion is not a thing. This text was likely a result of copying the variable name used in libuv. This commit updates the documentation to reference the correct Windows API call.

Refs: #31732

Checklist

• make -j4 test (UNIX), or vcbuild test (Windows) passes
• documentation is changed or added
• commit message follows commit guidelines

[juanarbol]

I suggest adding the RtlGetVersion() also on os.release() docs, as long as is basically doing the same, if RtlGetVersion() is not available the fallback is GetVersionExW()

[cjihrig]

I suggest adding the RtlGetVersion() also on os.release() docs, as long as is basically doing the same, if RtlGetVersion() is not available the fallback is GetVersionExW()

Honestly, I don't think Node should be calling out these implementation details. If anything, I'd direct users to the libuv docs. Not because the libuv docs are particularly good, but because Node uses libuv directly, and if implementation changes are made in libuv, the docs there are more likely to get updated properly.

[juanarbol]

Honestly, I don't think Node should be calling out these implementation details. If anything, I'd direct users to the libuv docs. Not because the libuv docs are particularly good, but because Node uses libuv directly, and if implementation changes are made in libuv, the docs there are more likely to get updated properly.

Yeah, I think you're right.

I'm +1 on this.