Skip to content
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: fix typo in fs writeSync param list #5984

Closed

Conversation

firedfox
Copy link
Contributor

@firedfox firedfox commented Apr 1, 2016

Pull Request check-list

  • Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with
    this change (including linting)?
  • Is the commit message formatted according to CONTRIBUTING.md?
  • If this change fixes a bug (or a performance problem), is a regression
    test (or a benchmark) included?
  • Is a documentation update included (if this change modifies
    existing APIs, or introduces new ones)?

Affected core subsystem(s)

  • doc

Description of change

Change buffer to data in the param list of fs.writeSync(fd, data[, position[, encoding]])

Rename `buffer` to `data` in param list of fs.writeSync(fd, data[, position[, encoding]])
@mscdex mscdex added doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system. labels Apr 1, 2016
@mscdex
Copy link
Contributor

mscdex commented Apr 1, 2016

LGTM

@jasnell
Copy link
Member

jasnell commented Apr 1, 2016

LGTM

jasnell pushed a commit that referenced this pull request Apr 1, 2016
Rename `buffer` to `data` in param list of
fs.writeSync(fd, data[, position[, encoding]])

PR-URL: #5984
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell
Copy link
Member

jasnell commented Apr 1, 2016

Landed in d939152

@MylesBorins
Copy link
Contributor

this change appears to make changes to copy that does not exist on v4 or v5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants