diff options
| author | DeeDeeG <DeeDeeG@users.noreply.github.com> | 2022-07-14 18:24:08 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-07-14 18:24:08 +0300 |
| commit | 48d4e3d0b2d621e1d4891d51aaf79a235f90c3dc (patch) | |
| tree | 1e472440841755b7fdb0d7521fe0c22dd322e696 /doc/api/dns.md | |
| parent | 34785657fba6e0eda2981e3fa6eb12c3b66509af (diff) | |
doc: revert anachronistic 'node:' module require()s in API history notes
Support for the 'node:' prefixed builtin module namespace was introduced
for `require()` expressions in Node v16.0.0, and backported to v14.18.0.
This was never supported in Node v15.x or chronologically older.
All of the current API history notes in the docs using 'node:' prefixed
module `require()`s happen to be documenting changes in Node versions
from before the time when support was first introduced.
This commit reverts those `require()`s in the history notes to be
un-prefixed. (They were incorrect as written; The prefixed `require()`s
would not work for those older Node versions.)
This change prevents the API history notes from inaccurately implying
'node:' prefixed builtin modules were introduced many Node versions ago,
or were `require()`-able with the 'node:' prefix in those Node versions.
Refs: https://github.com/nodejs/node/pull/35387
Refs: https://github.com/nodejs/node/pull/37246
Refs: https://github.com/nodejs/node/pull/42752
PR-URL: https://github.com/nodejs/node/pull/43768
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Diffstat (limited to 'doc/api/dns.md')
| -rw-r--r-- | doc/api/dns.md | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/doc/api/dns.md b/doc/api/dns.md index 0f43c7a1d72..660a8436941 100644 --- a/doc/api/dns.md +++ b/doc/api/dns.md @@ -832,7 +832,7 @@ added: v10.6.0 changes: - version: v15.0.0 pr-url: https://github.com/nodejs/node/pull/32953 - description: Exposed as `require('node:dns/promises')`. + description: Exposed as `require('dns/promises')`. - version: - v11.14.0 - v10.17.0 |