Skip to content

Conversation

@JacksonTian
Copy link
Contributor

@JacksonTianJacksonTian commented May 3, 2016

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

doc

Description of change

the buf.fill() and Buffer#indexOf don't have correctly anchors

@nodejs-github-botnodejs-github-bot added the doc Issues and PRs related to the documentations. label May 3, 2016
Copy link
Member

Choose a reason for hiding this comment

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

Could you change only the href here, not the text itself? I think that makes more sense.

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

hmmm, ok

@mscdexmscdex added the buffer Issues and PRs related to the buffer subsystem. label May 3, 2016
the `buf.fill()` and `Buffer#indexOf` don't have correctly anchors
@addaleax
Copy link
Member

LGTM

1 similar comment
@jasnell
Copy link
Member

LGTM

jasnell pushed a commit that referenced this pull request May 6, 2016
the `buf.fill()` and `Buffer#indexOf` don't have correctly anchors PR-URL: #6542 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]>
@jasnell
Copy link
Member

Landed in b8573d0. Thank you!

@jasnelljasnell closed this May 6, 2016
evanlucas pushed a commit that referenced this pull request May 17, 2016
the `buf.fill()` and `Buffer#indexOf` don't have correctly anchors PR-URL: #6542 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: James M Snell <[email protected]>
@MylesBorins
Copy link
Contributor

MylesBorins commented May 18, 2016

this is not landing cleanly on v4.x-staging

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bufferIssues and PRs related to the buffer subsystem.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@JacksonTian@addaleax@jasnell@MylesBorins@mscdex@nodejs-github-bot