Skip to content

Rework the async startegy - #36

Merged
Baptouuuu merged 4 commits into
mainfrom
async-rework
Aug 23, 2026
Merged

Rework the async startegy#36
Baptouuuu merged 4 commits into
mainfrom
async-rework

Conversation

@Baptouuuu

@Baptouuuu Baptouuuu commented Aug 23, 2026

Copy link
Copy Markdown
Member

Fix #31

Unlike mentioned in the issue the asAsync method as been added to the Config class instead of the Transport one in order to not duplicate the transformation logic.

@Baptouuuu Baptouuuu self-assigned this Aug 23, 2026
@Baptouuuu Baptouuuu added the enhancement New feature or request label Aug 23, 2026
@codecov

codecov Bot commented Aug 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 28.37838% with 53 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.78%. Comparing base (496fa78) to head (7ddc22a).

Files with missing lines Patch % Lines
src/Config.php 27.77% 13 Missing ⚠️
src/Transport/CircuitBreaker.php 0.00% 9 Missing ⚠️
src/Transport/ExponentialBackoff.php 0.00% 9 Missing ⚠️
src/Config/Async.php 0.00% 7 Missing ⚠️
src/Transport/Via.php 14.28% 6 Missing ⚠️
src/Transport/Curl.php 83.33% 3 Missing ⚠️
src/Transport/FollowRedirections.php 0.00% 3 Missing ⚠️
src/Transport/Logger.php 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main      #36      +/-   ##
============================================
- Coverage     91.90%   89.78%   -2.13%     
- Complexity      371      390      +19     
============================================
  Files            30       31       +1     
  Lines          2039     2104      +65     
============================================
+ Hits           1874     1889      +15     
- Misses          165      215      +50     

☔ View full report in Codecov by Harness.
📢 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.

@Baptouuuu
Baptouuuu merged commit ce030d0 into main Aug 23, 2026
20 of 22 checks passed
@Baptouuuu
Baptouuuu deleted the async-rework branch August 23, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rework the Async strategy

1 participant