Skip to content

Conversation

danbev
Copy link
Contributor

@danbev danbev commented Dec 3, 2020

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • 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 Dec 3, 2020
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@danbev
Copy link
Contributor Author

danbev commented Dec 9, 2020

Re-run of failing node-test-commit-arm-fanned ✔️

danbev added a commit that referenced this pull request Dec 9, 2020
PR-URL: #36365
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@danbev
Copy link
Contributor Author

danbev commented Dec 9, 2020

Landed in 068e0fe.

@danbev danbev closed this Dec 9, 2020
@danbev danbev deleted the process_methods_using branch December 9, 2020 11:59
Trott pushed a commit that referenced this pull request Dec 9, 2020
PR-URL: #36365
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
@Trott
Copy link
Member

Trott commented Dec 9, 2020

Landed in 068e0fe.

Re-Landed in 344a544 due to a force push to fix an unrelated issue.

targos pushed a commit that referenced this pull request Dec 21, 2020
PR-URL: #36365
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
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.

6 participants