Skip to content

Conversation

@christianlupus
Copy link
Collaborator

Closes #177

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

codecov bot commented May 21, 2022

Codecov Report

Merging #1014 (4e9eabe) into master (c707b67) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1014      +/-   ##
==========================================
- Coverage   32.30%   32.26%   -0.04%     
==========================================
  Files          32       32              
  Lines        1619     1621       +2     
==========================================
  Hits          523      523              
- Misses       1096     1098       +2     
Flag Coverage Δ
integration 5.24% <0.00%> (-0.01%) ⬇️
unittests 27.02% <0.00%> (-0.04%) ⬇️

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

Impacted Files Coverage Δ
lib/Service/RecipeService.php 0.00% <0.00%> (ø)

@github-actions
Copy link

github-actions bot commented May 21, 2022

Unit Test Results

     24 files       24 suites   8m 16s ⏱️
   138 tests    138 ✔️ 0 💤 0
1 656 runs  1 656 ✔️ 0 💤 0

Results for commit 4e9eabe.

♻️ This comment has been updated with latest results.

@christianlupus christianlupus marked this pull request as ready for review May 21, 2022 08:15
@christianlupus christianlupus merged commit 39fb5b1 into master May 21, 2022
@delete-merged-branch delete-merged-branch bot deleted the fix/177-too-long-recipe-names branch May 21, 2022 09:04
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.

Trying to import recipe from an unsupported site makes app unreachable

2 participants