Skip to content

doc: improved documentation for fs.unlink() #18843

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
dustinnewman wants to merge 5 commits into from
Closed

doc: improved documentation for fs.unlink() #18843

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b531ac0
doc: improved documentation for fs.unlink()
dustinnewman Feb 18, 2018
2272df5
fixup! address review comments
dustinnewman Feb 18, 2018
82b486c
fixup! Added function to remove directory.
dustinnewman Feb 18, 2018
8e2e47b
fixup! Addressed concerns.
dustinnewman Feb 18, 2018
55f37ae
fixup! wrapped at 80 characters
dustinnewman Feb 18, 2018
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
Prev Previous commit
Next Next commit
fixup! Added function to remove directory.
  • Loading branch information
@dustinnewman
dustinnewman committed Feb 18, 2018
commit 82b486cb9407e5f00b0017466dbf780dc6ef7e61
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 @@ -2760,7 +2760,7 @@ fs.unlink('path/file.txt', (err) => {
});
```

`fs.unlink()` will not work on a directory, empty or otherwise.
`fs.unlink()` will not work on a directory, empty or otherwise. To remove a directory, use `fs.rmdir()`.
Copy link
Copy Markdown
Member

@BridgeAR BridgeAR Feb 18, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be good to actually make this a link. Most functions are already listed but rmdir does not seem to be one of those. So please add the entry at the bottom of the file and write here:

[`fs.rmdir()`][]


```js
// Assuming that 'tmpDir' is a directory.
Expand Down