Skip to content

Conversation

@Jackyen
Copy link
Contributor

@JackyenJackyen commented Nov 22, 2017

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Nov 22, 2017
@MylesBorins
Copy link
Contributor

hey @Jackyen it looks like you have a spelling error in the commit message, would you be able to replace commom with common

@JackyenJackyen changed the title test: add commom.crashOnHandleRejectiontest: add common.crashOnHandleRejectionNov 22, 2017
@MylesBorinsMylesBorins added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Nov 22, 2017
@mscdexmscdex added the net Issues and PRs related to the net subsystem. label Nov 22, 2017
@jasnell
Copy link
Member

jasnell pushed a commit that referenced this pull request Nov 23, 2017
@jasnell
Copy link
Member

Landed in 6b56be2! Thank you and congrats on the PR to core!

@jasnelljasnell closed this Nov 23, 2017
MylesBorins pushed a commit that referenced this pull request Dec 12, 2017
MylesBorins pushed a commit that referenced this pull request Dec 12, 2017
@MylesBorinsMylesBorins mentioned this pull request Dec 12, 2017
MylesBorins pushed a commit that referenced this pull request Dec 19, 2017
gibfahn pushed a commit that referenced this pull request Dec 19, 2017
@gibfahngibfahn mentioned this pull request Dec 20, 2017
gibfahn pushed a commit that referenced this pull request Dec 20, 2017
@gibfahngibfahn mentioned this pull request Dec 20, 2017
@MylesBorinsMylesBorins mentioned this pull request Dec 20, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.netIssues and PRs related to the net subsystem.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Jackyen@MylesBorins@jasnell@mscdex@gibfahn@nodejs-github-bot