fix(node_http2.cc): remove duplicate codeblock
Original bug was introduced in here: https://github.com/nodejs/node/pull/55155/files#diff-33f026e43570112875cf4c8eab6743496f3aa014329611128e348ec23d6f771cL1303-R1313 Doesn't seem like that it actually leads to any problems, except extra memcpy
Review requested:
- [ ] @nodejs/http2
- [ ] @nodejs/net
CI: https://ci.nodejs.org/job/node-test-pull-request/63628/
@AVVS you need to update your commit message according to the requirements
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 88.50%. Comparing base (
1d01ad6) to head (6cefd81). Report is 1462 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #55915 +/- ##
==========================================
+ Coverage 88.01% 88.50% +0.49%
==========================================
Files 653 653
Lines 187735 187731 -4
Branches 35874 36184 +310
==========================================
+ Hits 165229 166149 +920
+ Misses 15693 14811 -882
+ Partials 6813 6771 -42
| Files with missing lines | Coverage Ξ | |
|---|---|---|
| src/node_http2.cc | 84.83% <ΓΈ> (+0.11%) |
:arrow_up: |
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
updated commit message
CI: https://ci.nodejs.org/job/node-test-pull-request/63635/
CI: https://ci.nodejs.org/job/node-test-pull-request/63696/
CI: https://ci.nodejs.org/job/node-test-pull-request/63705/
CI: https://ci.nodejs.org/job/node-test-pull-request/63766/
CI: https://ci.nodejs.org/job/node-test-pull-request/63887/
CI: https://ci.nodejs.org/job/node-test-pull-request/64008/
CI: https://ci.nodejs.org/job/node-test-pull-request/64022/
Landed in 131c33efa56ce7630bf9d25e5b8c3467b5f142ed