Skip to content

Conversation

@kgiusti
Copy link
Contributor

@kgiusti kgiusti commented Feb 12, 2021

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #1030 (a5f0ad7) into master (134b1ea) will increase coverage by 3.39%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1030      +/-   ##
==========================================
+ Coverage   79.02%   82.42%   +3.39%     
==========================================
  Files         112      111       -1     
  Lines       28553    27316    -1237     
==========================================
- Hits        22565    22514      -51     
+ Misses       5988     4802    -1186     
Impacted Files Coverage Δ
tests/parse_test.c 69.10% <ø> (ø)
src/parse.c 86.87% <100.00%> (-0.20%) ⬇️
src/iterator.c 89.29% <0.00%> (-3.88%) ⬇️
src/router_core/forwarder.c 93.04% <0.00%> (-0.80%) ⬇️
src/router_core/connections.c 88.81% <0.00%> (-0.45%) ⬇️
src/router_core/transfer.c 92.02% <0.00%> (-0.06%) ⬇️
src/adaptors/http2/http2_adaptor.c
src/adaptors/http2/http2_adaptor.h
src/adaptors/http2/http2_adaptor_none.c 0.00% <0.00%> (ø)
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 134b1ea...a5f0ad7. Read the comment docs.

Copy link
Contributor

@jiridanek jiridanek left a comment

Choose a reason for hiding this comment

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

I've checked the CI logs: Runtime warnings about shifts have disappeared. CI fails do not look to be related to this PR.

@asfgit asfgit merged commit e7e7189 into apache:master Mar 22, 2021
@kgiusti kgiusti deleted the DISPATCH-1949 branch December 14, 2021 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants