Skip to content

Conversation

addaleax
Copy link
Member

Refs: #30552 (review)

@vsemozhetbyt

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

@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Nov 28, 2019
@cjihrig
Copy link
Contributor

cjihrig commented Nov 28, 2019

Partial duplicate of #30662.

@addaleax
Copy link
Member Author

addaleax commented Nov 28, 2019

Ah, sorry. Git will rebase out your changes anyway (out of this PR) as they match 1:1, so I think this is still okay.

addaleax added a commit that referenced this pull request Nov 30, 2019
Refs: #30552 (review)

PR-URL: #30693
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
@addaleax
Copy link
Member Author

Landed in 0f42727

@addaleax addaleax closed this Nov 30, 2019
@addaleax addaleax deleted the cpp-readme-nits branch November 30, 2019 00:36
addaleax added a commit that referenced this pull request Nov 30, 2019
Refs: #30552 (review)

PR-URL: #30693
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
@BridgeAR BridgeAR mentioned this pull request Dec 3, 2019
targos pushed a commit that referenced this pull request Dec 5, 2019
Refs: #30552 (review)

PR-URL: #30693
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
@BethGriggs BethGriggs mentioned this pull request Dec 9, 2019
MylesBorins pushed a commit that referenced this pull request Dec 17, 2019
Refs: #30552 (review)

PR-URL: #30693
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
@BethGriggs BethGriggs mentioned this pull request Dec 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants