Skip to content

Use ntex error for connect#780

Open
fafhrd91 wants to merge 1 commit intomainfrom
error
Open

Use ntex error for connect#780
fafhrd91 wants to merge 1 commit intomainfrom
error

Conversation

@fafhrd91
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

❌ Patch coverage is 82.29665% with 37 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.56%. Comparing base (6c40ade) to head (6dc8307).

Files with missing lines Patch % Lines
ntex/src/client/response.rs 73.58% 14 Missing ⚠️
ntex/src/client/h2proto.rs 75.86% 7 Missing ⚠️
ntex/src/client/connector.rs 90.47% 4 Missing ⚠️
ntex/src/client/h1proto.rs 84.61% 4 Missing ⚠️
ntex/src/client/request.rs 87.87% 4 Missing ⚠️
ntex/src/client/pool.rs 50.00% 2 Missing ⚠️
ntex/src/http/error.rs 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #780      +/-   ##
==========================================
+ Coverage   90.52%   90.56%   +0.03%     
==========================================
  Files         239      239              
  Lines       33608    33702      +94     
==========================================
+ Hits        30425    30522      +97     
+ Misses       3183     3180       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fafhrd91 fafhrd91 force-pushed the error branch 3 times, most recently from d2994ef to c0b1d5a Compare March 1, 2026 09:42
@fafhrd91 fafhrd91 changed the title Error management impl Use ntex error for connect Mar 1, 2026
@fafhrd91 fafhrd91 force-pushed the error branch 6 times, most recently from bbe0575 to ab35a06 Compare March 7, 2026 18:30
@fafhrd91 fafhrd91 force-pushed the error branch 4 times, most recently from ed25ff3 to fd2ba29 Compare March 14, 2026 15:59
@fafhrd91 fafhrd91 force-pushed the error branch 8 times, most recently from 1ed4c71 to 9c7da82 Compare March 19, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant