Skip to content

doc: fix typo in fs writeSync param list#5984

Closed
firedfox wants to merge 1 commit intonodejs:masterfrom
firedfox:fix-typo-in-fs-writeSync-param-list
Closed

doc: fix typo in fs writeSync param list#5984
firedfox wants to merge 1 commit intonodejs:masterfrom
firedfox:fix-typo-in-fs-writeSync-param-list

Conversation

@firedfox
Copy link
Copy Markdown
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
Copy Markdown
Contributor

mscdex commented Apr 1, 2016

LGTM

@jasnell
Copy link
Copy Markdown
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
Copy Markdown
Member

jasnell commented Apr 1, 2016

Landed in d939152

@MylesBorins
Copy link
Copy Markdown
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