Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] tests/connection-abort merge #440

Merged
merged 3 commits into from
Nov 4, 2023
Merged

Conversation

thibaultcha
Copy link
Member

No description provided.

Copy link

codecov bot commented Nov 4, 2023

Codecov Report

Merging #440 (d9f36e3) into main (de9eb4c) will decrease coverage by 0.00118%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@                 Coverage Diff                 @@
##                main        #440         +/-   ##
===================================================
- Coverage   90.11905%   90.11787%   -0.00119%     
===================================================
  Files             46          46                 
  Lines           8400        8399          -1     
===================================================
- Hits            7570        7569          -1     
  Misses           830         830                 
Files Coverage Δ
src/common/ngx_wasm_socket_tcp_readers.c 86.63594% <ø> (-0.06131%) ⬇️

@thibaultcha thibaultcha force-pushed the tests/connection-abort branch 2 times, most recently from 5bbb08f to 6e03a57 Compare November 4, 2023 17:10
thibaultcha and others added 3 commits November 4, 2023 10:14
And reorder patches from bottom to top of the file.
It fails when testing aborted connection behavior
with dispatch calls.
@thibaultcha thibaultcha force-pushed the tests/connection-abort branch from 6e03a57 to d9f36e3 Compare November 4, 2023 17:14
@thibaultcha thibaultcha merged commit 689a460 into main Nov 4, 2023
32 checks passed
@thibaultcha thibaultcha deleted the tests/connection-abort branch November 4, 2023 19:54
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.

2 participants