doc: add util.types.isModuleNamespaceObject() #20616
Conversation
nodejs-github-bot
added
doc
|
|
||
| Returns `true` if the value is an instance of a [Module Namespace Object][]. | ||
|
|
||
| For example: |
There was a problem hiding this comment.
as much as i hate that line, every other one has that line.
There was a problem hiding this comment.
I can open another or after this to remove it from all of the docs at once.
doc/api/util.md
Outdated
|
|
||
| ```js | ||
| import('./a.js').then((ns) => { | ||
| util.types.isModuleNamespaceObject(ns); // Returns true |
There was a problem hiding this comment.
(Not necessarily true if there is a then export.)
I’d use an import * as ns example.
There was a problem hiding this comment.
it won't pass lint if i do that, i'd have to ignore the block
There was a problem hiding this comment.
It seems the comment is a bit confusing: as this is not just a REPL command, this line does not return in the usual sense, but the function returns Promise resolved to undefined. Maybe we should add return and change the comment into Resolves...?
There was a problem hiding this comment.
@devsnek Then IMO we should ignore the block.
There was a problem hiding this comment.
does it look okay at this point?
|
Node.js Collaborators, please, add 👍 here if you approve fast-tracking. |
devsnek
force-pushed
the
doc/util-types-ismodulenamespaceobject
branch
from
326cfa2
to
19c4f9b
Compare
|
landed in f1cdd77 |
PR-URL: #20616 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Guy Bedford <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
devsnek
removed
the
fast-track
PR-URL: #20616 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Guy Bedford <[email protected]> Reviewed-By: Vse Mozhet Byt <[email protected]> Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
Checklist