Skip to content

Use middlewares instead of the decorators#3

Merged
olivermack merged 2 commits intomainfrom
feat/guzzle-middlewares
Mar 2, 2025
Merged

Use middlewares instead of the decorators#3
olivermack merged 2 commits intomainfrom
feat/guzzle-middlewares

Conversation

@olivermack
Copy link
Owner

In favor of portability the lib should provide guzzle middlewares instead client decorators so that other guzzle handler stacks could possibly make use of the individual middlewares.

omack added 2 commits March 2, 2025 22:06
…d of dedicate decorators; a subclass of PSR Request is now carrying the enhanced response around
@github-actions
Copy link

github-actions bot commented Mar 2, 2025

Code Coverage

Package Line Rate Health
Duzzle.php 100%
DuzzleBuilder.php 100%
DuzzleResponse.php 100%
Serialization/CarriesResponseTrait.php 100%
Serialization/DefaultSerializerFactory.php 100%
Serialization/SerializationHandler.php 91%
Serialization/SerializationMiddleware.php 100%
Validation/DefaultStrategyCollectionFactory.php 100%
Validation/DefaultValidatorFactory.php 100%
Validation/Strategy/BlockingValidationStrategy.php 100%
Validation/Strategy/InformativeValidationStrategy.php 100%
Validation/Strategy/NoopValidationStrategy.php 100%
Validation/Strategy/ValidationStrategyCollection.php 100%
Validation/ValidationHandler.php 100%
Validation/ValidationMiddleware.php 100%
Summary 98% (215 / 220)

Minimum allowed line rate is 90%

@olivermack olivermack merged commit bb4c3f8 into main Mar 2, 2025
6 checks passed
@olivermack olivermack deleted the feat/guzzle-middlewares branch March 2, 2025 21:24
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