-
-
Notifications
You must be signed in to change notification settings - Fork 387
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat!: Remove deprecated litestar.middleware.exceptions
module and deprecated params of internal ExceptionHandlerMiddleware
#3435
feat!: Remove deprecated litestar.middleware.exceptions
module and deprecated params of internal ExceptionHandlerMiddleware
#3435
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v3.0 #3435 +/- ##
==========================================
- Coverage 98.25% 98.24% -0.01%
==========================================
Files 322 323 +1
Lines 14672 14455 -217
Branches 2332 2295 -37
==========================================
- Hits 14416 14202 -214
- Misses 117 118 +1
+ Partials 139 135 -4 ☔ View full report in Codecov by Sentry. |
dd8c727
to
5c13610
Compare
4dc78a8
to
81a323f
Compare
1882376
to
ea2545c
Compare
…e` (#3393) * Remove 'app' parameter from `.to_asgi_response`
4dbe8c6
to
e67dcc7
Compare
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/3435 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
even @JacobCoffee is not this slow in reviewing
Description
litestar.middleware.exceptions
app
param ofExceptionHandlerMiddleware
exception_handlers
param ofExceptionHandlerMiddleware
Closes