Skip to content

Conversation

@jasnell
Copy link
Member

@jasnell jasnell commented Sep 5, 2017

Improve http2 coverage through refactoring and tests

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

http2, test

Improve http2 coverage through refactoring and tests
@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. http2 Issues or PRs related to the http2 subsystem. labels Sep 5, 2017
@jasnell jasnell requested a review from mcollina September 5, 2017 22:04
@BridgeAR
Copy link
Member

BridgeAR commented Sep 9, 2017

CI https://ci.nodejs.org/job/node-test-pull-request/10015/

[I posted the wrong one first]

@apapirovski
Copy link
Contributor

Would be nice to see this get reviewed & merged.

Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

LGTM

@mcollina
Copy link
Member

This does not land cleanly, it needs a rebase.

jasnell added a commit that referenced this pull request Sep 13, 2017
Improve http2 coverage through refactoring and tests

PR-URL: #15210
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
@jasnell
Copy link
Member Author

jasnell commented Sep 13, 2017

It just needed a -3 to land. Landed in ad3d899

@jasnell jasnell closed this Sep 13, 2017
addaleax pushed a commit to addaleax/ayo that referenced this pull request Sep 17, 2017
Improve http2 coverage through refactoring and tests

PR-URL: nodejs/node#15210
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
jasnell added a commit that referenced this pull request Sep 20, 2017
Improve http2 coverage through refactoring and tests

PR-URL: #15210
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Qard pushed a commit to Qard/ayo that referenced this pull request Sep 21, 2017
Improve http2 coverage through refactoring and tests

PR-URL: nodejs/node#15210
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
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++. http2 Issues or PRs related to the http2 subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants