Skip to content

Don't convert deprecations to exceptions#19933

Open
kamil-tekiela wants to merge 1 commit intophpmyadmin:masterfrom
kamil-tekiela:Don't-convert-deprecations-to-exceptions
Open

Don't convert deprecations to exceptions#19933
kamil-tekiela wants to merge 1 commit intophpmyadmin:masterfrom
kamil-tekiela:Don't-convert-deprecations-to-exceptions

Conversation

@kamil-tekiela
Copy link
Contributor

Alternative to #19932

Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
@codecov
Copy link

codecov bot commented Nov 11, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 62.25%. Comparing base (6d3d61f) to head (af75608).
⚠️ Report is 50 commits behind head on master.

Files with missing lines Patch % Lines
src/Error/ErrorHandler.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #19933      +/-   ##
============================================
+ Coverage     62.21%   62.25%   +0.04%     
+ Complexity    16170    16164       -6     
============================================
  Files           675      674       -1     
  Lines         60222    60198      -24     
============================================
+ Hits          37467    37478      +11     
+ Misses        22755    22720      -35     
Flag Coverage Δ
dbase-extension 62.22% <0.00%> (+0.04%) ⬆️
unit-8.2-ubuntu-latest 62.17% <0.00%> (+0.02%) ⬆️
unit-8.3-ubuntu-latest ?
unit-8.4-ubuntu-latest 62.19% <0.00%> (+0.05%) ⬆️
unit-8.5-ubuntu-latest 62.17% <0.00%> (-0.01%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but they will display ?

@kamil-tekiela
Copy link
Contributor Author

Currently this is how it shows up:
In production:
image
In development:
image

With this PR this is how it will be:
In production:
image
In development:
image

@williamdes
Copy link
Member

Thanks, I better understand this PR now
Can you make "In production" deprecations go away ?
Users do not need to see them, it creates endless discussions and confusion

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