Skip to content

Conversation

@addaleax
Copy link
Member

Various improvements to the Buffer docs.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

Various improvements to the Buffer docs.
@nodejs-github-botnodejs-github-bot added buffer Issues and PRs related to the buffer subsystem. doc Issues and PRs related to the documentations. labels Mar 4, 2020
@addaleax
Copy link
MemberAuthor

@jasnell I’ve updated this based on feedback I’ve received in private, feel free to take another look :)

@addaleaxaddaleaxforce-pushed the buffer-doc-rewrite branch from cb995aa to 8cc28abCompareMarch 7, 2020 11:42
@addaleax
Copy link
MemberAuthor

@nodejs/documentation

@jasnell
Copy link
Member

Still LGTM

@addaleaxaddaleax added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Mar 11, 2020
@addaleax
Copy link
MemberAuthor

Landed in c41e360

addaleax added a commit that referenced this pull request Mar 11, 2020
Various improvements to the Buffer docs. PR-URL: #32086 Reviewed-By: James M Snell <[email protected]>
@addaleaxaddaleax deleted the buffer-doc-rewrite branch March 11, 2020 16:07
MylesBorins pushed a commit that referenced this pull request Mar 11, 2020
Various improvements to the Buffer docs. PR-URL: #32086 Reviewed-By: James M Snell <[email protected]>
@MylesBorinsMylesBorins mentioned this pull request Mar 12, 2020
targos pushed a commit that referenced this pull request Apr 22, 2020
Various improvements to the Buffer docs. PR-URL: #32086 Reviewed-By: James M Snell <[email protected]>
@targostargos mentioned this pull request Apr 22, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.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.

3 participants

@addaleax@jasnell@nodejs-github-bot