Skip to content
This repository was archived by the owner on Jul 6, 2018. It is now read-only.

http2: fix http2-server-startup#149

Closed
jasnell wants to merge 1 commit intonodejs:masterfrom
jasnell:fix-http2-server-startup
Closed

http2: fix http2-server-startup#149
jasnell wants to merge 1 commit intonodejs:masterfrom
jasnell:fix-http2-server-startup

Conversation

@jasnell
Copy link
Copy Markdown
Member

@jasnell jasnell commented Jun 6, 2017

9b6b067 broke the test... this fixes it

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)

mcollina

This comment was marked as off-topic.

sebdeckers

This comment was marked as off-topic.

jasnell added a commit that referenced this pull request Jun 7, 2017
9b6b067 broke the test

PR-URL: #149
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Sebastiaan Deckers <sebdeckers83@gmail.com>
@jasnell
Copy link
Copy Markdown
Member Author

jasnell commented Jun 7, 2017

Landed!

(and no worries @sebdeckers! :-) ...)

@jasnell jasnell closed this Jun 7, 2017
jasnell added a commit to jasnell/http2-1 that referenced this pull request Jun 22, 2017
9b6b067 broke the test

PR-URL: nodejs#149
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Sebastiaan Deckers <sebdeckers83@gmail.com>
jasnell added a commit to jasnell/http2-1 that referenced this pull request Jul 10, 2017
9b6b067 broke the test

PR-URL: nodejs#149
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Sebastiaan Deckers <sebdeckers83@gmail.com>
jasnell added a commit to jasnell/http2-1 that referenced this pull request Jul 14, 2017
9b6b067 broke the test

PR-URL: nodejs#149
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Sebastiaan Deckers <sebdeckers83@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants