Skip to content

Conversation

@italoacasas
Copy link

Checklist
  • documentation is changed or added
  • the commit message follows commit guidelines
Affected core subsystem(s)

doc

Description of change

Fixing typo in the documentation.

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label Jun 21, 2016
@italoacasasitaloacasas changed the title Fixing link in streams docsdoc: fix link in the stream docJun 21, 2016
@mscdexmscdex added the stream Issues and PRs related to the stream subsystem. label Jun 21, 2016
@thefourtheye
Copy link
Contributor

Can you fix the spelling as well? I think it is supposed to be "API for Stream Implementers".

cc @nodejs/documentation

@addaleax
Copy link
Member

@thefourtheye is right, it should read “Implementers”.

If you want to take a look at the others from #6947 (comment), you can to that too (I’d recommend rebasing against master first now that #7336 has landed).

@italoacasas
Copy link
Author

Ok i'll check the others and fix the typo.

@italoacasas
Copy link
Author

italoacasas commented Jun 22, 2016

This is done.

@addaleax
Copy link
Member

LGTM

1 similar comment
@thefourtheye
Copy link
Contributor

LGTM

@addaleax
Copy link
Member

Landed in 79f961a

addaleax pushed a commit that referenced this pull request Jun 23, 2016
fixing typo fixing broken links PR-URL: #7347 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Fishrock123 pushed a commit that referenced this pull request Jun 27, 2016
fixing typo fixing broken links PR-URL: #7347 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
@Fishrock123Fishrock123 mentioned this pull request Jun 27, 2016
Fishrock123 pushed a commit that referenced this pull request Jul 5, 2016
fixing typo fixing broken links PR-URL: #7347 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]>
@Fishrock123Fishrock123 mentioned this pull request Jul 5, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.streamIssues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@italoacasas@thefourtheye@addaleax@mscdex@MylesBorins@nodejs-github-bot