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
@Rastusik Rastusik force-pushed the platform_idenpendent_streamed_response_processing branch from 388c400 to 371e54a Compare May 3, 2021 11:01
@k911 k911 added this to In progress in Development via automation May 3, 2021
@k911 k911 added this to Needs triage in Bugs via automation May 3, 2021
@k911 k911 moved this from Needs triage to High priority in Bugs May 3, 2021
@k911 k911 self-assigned this May 3, 2021
@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
Development automation moved this from In progress to Done May 5, 2021
Bugs automation moved this from High priority to Closed 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
Bugs
  
Closed
Development
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants