error-handler icon indicating copy to clipboard operation
error-handler copied to clipboard

Add solution providers

Open xepozz opened this issue 1 year ago • 9 comments

Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Fixed issues #108

xepozz avatar Jul 18 '24 09:07 xepozz

Looks cool! But I suggest do it in separate package.

vjik avatar Aug 05 '24 06:08 vjik

Looks cool! But I suggest do it in separate package.

All the rest implementation, but it's base support for all of them

xepozz avatar Oct 14 '24 16:10 xepozz

@xepozz what's missing here?

samdark avatar Dec 21 '24 20:12 samdark

I think I lost some files in the original codebase, some of them already moved to bridge for error handling 😁

xepozz avatar Dec 22 '24 11:12 xepozz

Possible to update it?

samdark avatar Dec 23 '24 07:12 samdark

Codecov Report

:x: Patch coverage is 12.50000% with 7 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 79.76%. Comparing base (44f49fa) to head (3a8bcc7). :warning: Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/Solution/FriendlyExceptionSolution.php 0.00% 4 Missing :warning:
src/Renderer/HtmlRenderer.php 25.00% 3 Missing :warning:
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #127      +/-   ##
============================================
- Coverage     80.55%   79.76%   -0.80%     
- Complexity      210      213       +3     
============================================
  Files            19       20       +1     
  Lines           679      687       +8     
============================================
+ Hits            547      548       +1     
- Misses          132      139       +7     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Dec 24 '24 19:12 codecov[bot]

@xepozz would you please address static analysis issues? https://github.com/yiisoft/error-handler/actions/runs/16853615512/job/47743450768?pr=127

samdark avatar Aug 10 '25 08:08 samdark

@vjik noted that HTML doesn't suit console handler well. That is true.

samdark avatar Aug 10 '25 18:08 samdark

SolutionProvider's are in use only in the html renderer.

xepozz avatar Aug 11 '25 06:08 xepozz