Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

Trott
Copy link
Member

@Trott Trott commented Dec 17, 2020

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 doc Issues and PRs related to the documentations. worker Issues and PRs related to Worker support. labels Dec 17, 2020
Copy link
Member

@Lxxyx Lxxyx left a comment

Choose a reason for hiding this comment

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

LGTM. It's easier to understand.

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

PR-URL: nodejs#36545
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@Trott Trott merged commit b5fdb9e into nodejs:master Dec 21, 2020
@Trott
Copy link
Member Author

Trott commented Dec 21, 2020

Landed in b5fdb9e

@Trott Trott deleted the wt-revise branch December 21, 2020 13:45
targos pushed a commit that referenced this pull request Dec 21, 2020
PR-URL: #36545
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. worker Issues and PRs related to Worker support.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants