Skip to content

process: invokes setImmediate on uv check phase#406

Merged
yorkie merged 7 commits intomasterfrom
uv/handle
Nov 9, 2018
Merged

process: invokes setImmediate on uv check phase#406
yorkie merged 7 commits intomasterfrom
uv/handle

Conversation

@legendecas
Copy link
Copy Markdown
Contributor

  • npm test passes
  • tests and/or benchmarks are included

Also enables uv check/prepare handles in libtuv.

@legendecas legendecas requested review from qile222 and yorkie November 8, 2018 06:32
@legendecas legendecas changed the title process: setImmediate shall be invoked on uv check phase process: invokes setImmediate on uv check phase Nov 8, 2018
Copy link
Copy Markdown
Member

@yorkie yorkie left a comment

Choose a reason for hiding this comment

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

Leaves comments, could you please add some benchmark, too?

@yorkie yorkie added the minor minor changes label Nov 8, 2018
@yorkie
Copy link
Copy Markdown
Member

yorkie commented Nov 8, 2018

Labeled with minor, too.

@yorkie
Copy link
Copy Markdown
Member

yorkie commented Nov 9, 2018

Ping @legendecas, still leaves a #406 (comment) from @lolBig.

@legendecas legendecas force-pushed the uv/handle branch 2 times, most recently from 44eb89d to e83a01b Compare November 9, 2018 06:57
@yorkie yorkie merged commit 412fdc9 into master Nov 9, 2018
@yorkie yorkie deleted the uv/handle branch November 9, 2018 15:44
@yorkie
Copy link
Copy Markdown
Member

yorkie commented Nov 20, 2018

Removed the label wont-land-on-v0.11.x because #417 fixed.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants