From fb36266f28379fc7270a3d3e7423296e59e022b1 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Sat, 7 Mar 2020 17:35:00 -0800 Subject: [PATCH] doc: remove personal pronoun usage in fs.md MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Per our style guide, avoid personal pronouns (I, you, we, etc.) in reference documentation. PR-URL: https://github.com/nodejs/node/pull/32142 Reviewed-By: Tobias Nießen Reviewed-By: Ruben Bridgewater Reviewed-By: Trivikram Kamat --- doc/api/fs.md | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/doc/api/fs.md b/doc/api/fs.md index 24daaaaa53c3df..a70f56f3c922af 100644 --- a/doc/api/fs.md +++ b/doc/api/fs.md @@ -1710,9 +1710,9 @@ By default, the stream will not emit a `'close'` event after it has been destroyed. This is the opposite of the default for other `Readable` streams. Set the `emitClose` option to `true` to change this behavior. -By providing the `fs` option it is possible to override the corresponding `fs` -implementations for `open`, `read` and `close`. When providing the `fs` option, -you must override `open`, `close` and `read`. +By providing the `fs` option, it is possible to override the corresponding `fs` +implementations for `open`, `read`, and `close`. When providing the `fs` option, +overrides for `open`, `read`, and `close` are required. ```js const fs = require('fs'); @@ -1805,8 +1805,8 @@ Set the `emitClose` option to `true` to change this behavior. By providing the `fs` option it is possible to override the corresponding `fs` implementations for `open`, `write`, `writev` and `close`. Overriding `write()` without `writev()` can reduce performance as some optimizations (`_writev()`) -will be disabled. When providing the `fs` option, you must override `open`, -`close` and at least one of `write` and `writev`. +will be disabled. When providing the `fs` option, overrides for `open`, +`close`, and at least one of `write` and `writev` are required. Like [`ReadStream`][], if `fd` is specified, [`WriteStream`][] will ignore the `path` argument and will use the specified file descriptor. This means that no @@ -4256,8 +4256,8 @@ in that they provide an object oriented API for working with files. If a `FileHandle` is not closed using the `filehandle.close()` method, it might automatically close the file descriptor and will emit a process warning, thereby helping to prevent memory leaks. -Please do not rely on this behavior in your code because it is unreliable and -your file may not be closed. Instead, always explicitly close `FileHandle`s. +Please do not rely on this behavior because it is unreliable and +the file may not be closed. Instead, always explicitly close `FileHandle`s. Node.js may change this behavior in the future. Instances of the `FileHandle` object are created internally by the