Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

doc: fix minor typos and grammar in fs docs - #7431

Closed
linclark wants to merge 1 commit into
nodejs:masterfrom
linclark:fs-docs
Closed

linclark wants to merge 1 commit into
nodejs:masterfrom
linclark:fs-docs

Conversation

@linclark

@linclark linclark commented Apr 9, 2014

Copy link
Copy Markdown

No description provided.

@Nodejs-Jenkins

Copy link
Copy Markdown

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

The following commiters were not found in the CLA:

  • linclark

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

@linclark

linclark commented Apr 9, 2014

Copy link
Copy Markdown
Author

CLA signed.

@a0viedo

a0viedo commented Dec 10, 2014

Copy link
Copy Markdown
Member

Hey @linclark could you re-submit this PR to https://github.com/iojs/io.js? Someone could adopt it for you if you are not interested.

@geek

geek commented Jan 23, 2015

Copy link
Copy Markdown
Member

Reviewed, this looks good to me. Thanks @linclark. Interested in resubmitting the PR? The patch is a little out of date and doesn't pull in nicely with the current master.

@cjihrig

cjihrig commented Jan 23, 2015

Copy link
Copy Markdown

LGTM

@bhollan

bhollan commented Jul 24, 2015

Copy link
Copy Markdown

@linclark @Nodejs-Jenkins @a0viedo @geek @cjihrig

Who can get this closed and/or figure out why this isn't in the docs?

@a0viedo

a0viedo commented Jul 24, 2015

Copy link
Copy Markdown
Member

ping @jasnell maybe?

@jasnell

jasnell commented Jul 24, 2015

Copy link
Copy Markdown
Member

@linclark ... will be working on landing more doc commits next week. Will include this!

@jasnell jasnell self-assigned this Jul 24, 2015
@jasnell jasnell added the doc label Aug 14, 2015
@jasnell

jasnell commented Aug 14, 2015

Copy link
Copy Markdown
Member

Closed here. Opened a new PR on nodejs/node. Will land this in a batch with other documentation updates

@jasnell jasnell closed this Aug 14, 2015
jasnell added a commit to jasnell/node that referenced this pull request Aug 26, 2015
jasnell added a commit to nodejs/node that referenced this pull request Aug 26, 2015
per: nodejs/node-v0.x-archive#7431

originally submitted by @linclark

Reviewed By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
PR-URL: #2378
jasnell added a commit to nodejs/node that referenced this pull request Aug 26, 2015
per: nodejs/node-v0.x-archive#7431

originally submitted by @linclark

Reviewed By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
PR-URL: #2378
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants