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

fix(static-server): Fix unset public dir path in "AdvancedStaticFilesServer" #52

Merged
merged 1 commit into from
Apr 28, 2019

Conversation

k911
Copy link
Owner

@k911 k911 commented Apr 28, 2019

A bug caused that "AdvancedStaticFilesServer" was not tagged as bootable service, what resulted in
unsed public directory path.

…Server"

A bug caused that "AdvancedStaticFilesServer" was not tagged as bootable service, what resulted in
unsed public directory path.
@k911 k911 force-pushed the fix/static-files-serving branch from 3d2b317 to 17efbc1 Compare April 28, 2019 21:12
@codeclimate
Copy link

codeclimate bot commented Apr 28, 2019

Code Climate has analyzed commit 17efbc1 and detected 0 issues on this pull request.

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

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

View more on Code Climate.

@codecov-io
Copy link

codecov-io commented Apr 28, 2019

Codecov Report

Merging #52 into develop will increase coverage by 0.73%.
The diff coverage is 83.33%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop      #52      +/-   ##
=============================================
+ Coverage      83.82%   84.55%   +0.73%     
- Complexity       463      464       +1     
=============================================
  Files             67       67              
  Lines           1434     1431       -3     
=============================================
+ Hits            1202     1210       +8     
+ Misses           232      221      -11
Impacted Files Coverage Δ Complexity Δ
src/Client/Http.php 0% <ø> (ø) 1 <0> (ø) ⬇️
src/Client/HttpClient.php 74.44% <100%> (+0.28%) 33 <0> (+1) ⬆️
...ony/Bundle/DependencyInjection/SwooleExtension.php 83.17% <80%> (+1.19%) 33 <0> (ø) ⬇️
...erver/RequestHandler/AdvancedStaticFilesServer.php 88.46% <0%> (+34.61%) 11% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be304d5...17efbc1. Read the comment docs.

@k911 k911 merged commit 4ef8cb5 into develop Apr 28, 2019
@k911 k911 deleted the fix/static-files-serving branch April 28, 2019 21:23
k911 pushed a commit that referenced this pull request Apr 28, 2019
##  (2019-04-28)

* fix(static-server): Fix unset public dir path in "AdvancedStaticFilesServer" (#52) ([4ef8cb5](4ef8cb5)), closes [#52](#52)
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