Skip to content
This repository has been archived by the owner on Jan 17, 2022. It is now read-only.

feat(response-processor): hiding Swoole StreamedResponse support from Symfony #509

Conversation

Rastusik
Copy link
Contributor

@Rastusik Rastusik commented May 3, 2021

Implementation according to this conversation: #300 (comment)

@Rastusik Rastusik requested a review from k911 as a code owner May 3, 2021 09:41
@codecov
Copy link

codecov bot commented May 3, 2021

Codecov Report

Merging #509 (371e54a) into develop (8793695) will increase coverage by 0.45%.
The diff coverage is 96.87%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #509      +/-   ##
=============================================
+ Coverage      85.61%   86.06%   +0.45%     
- Complexity       661      663       +2     
=============================================
  Files             97       97              
  Lines           2078     2089      +11     
=============================================
+ Hits            1779     1798      +19     
+ Misses           299      291       -8     
Impacted Files Coverage Δ Complexity Δ
...ymfony/HttpFoundation/StreamedResponseListener.php 94.11% <90.90%> (+10.78%) 7.00 <1.00> (+2.00)
...tion/CompilerPass/StreamedResponseListenerPass.php 100.00% <100.00%> (+25.00%) 3.00 <0.00> (+1.00)
...mfony/HttpFoundation/ResponseProcessorInjector.php 100.00% <100.00%> (ø) 2.00 <2.00> (?)
...ge/Symfony/HttpKernel/HttpKernelRequestHandler.php 100.00% <100.00%> (ø) 4.00 <0.00> (ø)
...y/HttpFoundation/NoOpStreamedResponseProcessor.php 100.00% <0.00%> (+16.66%) 3.00% <0.00%> (ø%)

@Rastusik Rastusik force-pushed the platform_idenpendent_streamed_response_processing branch from c132552 to 388c400 Compare May 3, 2021 10:13
@k911
Copy link
Owner

k911 commented May 3, 2021

@Rastusik good work here, tomorrow I'll debug why some tests are failing here, probably unrelated to these changes

@codeclimate
Copy link

codeclimate bot commented May 3, 2021

Code Climate has analyzed commit 371e54a and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 97.1% (50% is the threshold).

This pull request will bring the total coverage in the repository to 86.1% (0.5% change).

View more on Code Climate.

@k911 k911 merged commit ebe8077 into k911:develop May 5, 2021
swoole-bundle-bot added a commit that referenced this pull request May 5, 2021
* ci: remove php 74 latest from build ([3e4c8f1](3e4c8f1))
* ci(docker): upgrade images used as dependencies ([8793695](8793695))
* chore(composer): lock php-cs-fixer to ^2.19.0 ([9e40296](9e40296))
* chore(composer): upgrade dependencies ([903e484](903e484))
* chore(deps): upgrade deps ([4718a82](4718a82))
* chore(docker): upgrade dependencies ([27393f8](27393f8))
* feat(response-processor): hiding Swoole StreamedResponse support from Symfony (#509) ([ebe8077](ebe8077)), closes [#509](#509)
@Rastusik Rastusik deleted the platform_idenpendent_streamed_response_processing branch May 5, 2021 12:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants