Skip to content

doc: remove "Synchronously" for fsPromises.rm() #35568

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.

Sign up for GitHub

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

Closed
Trott wants to merge 1 commit into from
Closed

doc: remove "Synchronously" for fsPromises.rm() #35568

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
doc: remove "Synchronously" for fsPromises.rm() 
Because it's a promise, fsPromises.rm() is asynchronous. That is
implicit in every other fsPromises entry, so this removes
"Synchronously" rather than replacing it with "Asynchronously".
  • Loading branch information
@Trott
Trott committed Oct 9, 2020
commit c834804792d79a2c4eaba433b1e457b06e1a51ac
2 changes: 1 addition & 1 deletion doc/api/fs.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5567,7 +5567,7 @@ added: REPLACEME
retries. This option is ignored if the `recursive` option is not `true`.
**Default:** `100`.

Synchronously removes files and directories (modeled on the standard POSIX `rm`
Removes files and directories (modeled on the standard POSIX `rm`
utility). Resolves the `Promise` with no arguments on success.

### `fsPromises.stat(path[, options])`
Expand Down