Skip to content

Conversation

@christianlupus
Copy link
Collaborator

This should make the dependencies more stable

@christianlupus christianlupus marked this pull request as draft July 21, 2022 20:40
@github-actions
Copy link

github-actions bot commented Jul 21, 2022

Unit Test Results

     36 files       36 suites   10m 45s ⏱️
   254 tests    254 ✔️ 0 💤 0
3 048 runs  3 048 ✔️ 0 💤 0

Results for commit 9d0d492.

♻️ This comment has been updated with latest results.

Signed-off-by: Christian Wolf <[email protected]>
@codecov
Copy link

codecov bot commented Jul 22, 2022

Codecov Report

Merging #1093 (9d0d492) into master (5d1d09c) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1093   +/-   ##
=======================================
  Coverage   45.38%   45.38%           
=======================================
  Files          53       53           
  Lines        2069     2069           
=======================================
  Hits          939      939           
  Misses       1130     1130           
Flag Coverage Δ
integration 2.17% <0.00%> (ø)
migration 4.49% <0.00%> (ø)
unittests 38.71% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
lib/Controller/RecipeController.php 100.00% <ø> (ø)
lib/Db/RecipeDb.php 0.00% <ø> (ø)
lib/Helper/HtmlToDomParser.php 95.16% <0.00%> (ø)
lib/Helper/ImageService/ImageFileHelper.php 100.00% <ø> (ø)
lib/Helper/ImageService/ThumbnailFileHelper.php 97.05% <ø> (ø)
lib/Migration/Version000000Date20210701093123.php 90.00% <ø> (ø)
lib/Search/Provider.php 0.00% <ø> (ø)
lib/Service/ImageService.php 100.00% <ø> (ø)
lib/Service/RecipeService.php 0.00% <ø> (ø)

@christianlupus christianlupus marked this pull request as ready for review July 22, 2022 08:42
@christianlupus christianlupus merged commit 74b0157 into master Jul 22, 2022
@christianlupus christianlupus deleted the fix/composer-lock branch July 22, 2022 08:44
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