-
Notifications
You must be signed in to change notification settings - Fork 29.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: add added:
information for fs
#6717
Conversation
ping @nodejs/documentation … anybody minds reviewing this? ;) |
LGTM. Didn't have a look at the rendering, but couldn't find anything that breaks. Who is doing all that research for history and how? By hand? |
@eljefedelrodeodeljefe For the “who”… see the issue linked above, so far that’s me, @bengl and @Trott, but you’re definitely invited to join the party. And the method depends, but |
Yeah, I was thinking so. Then please have my shout-outs! When I see some I add some... good work! |
Landed in ba10ea8 |
Ref: nodejs#6578 PR-URL: nodejs#6717 Reviewed-By: Robert Lindstaedt <robert.lindstaedt@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
fs.access and fs.accessSync were added to Node v0.11.15 via 2944934 PR-URL: nodejs#7299 Reviewed-By: Anna Henningsen <anna@addaleax.net> Ref: nodejs#6717 Ref: nodejs#6578
Checklist
Affected core subsystem(s)
doc
Description of change
Ref: #6578