-
-
Notifications
You must be signed in to change notification settings - Fork 402
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
fix(warnings): Do not warn for default handlers #3569
fix(warnings): Do not warn for default handlers #3569
Conversation
Added a test that fails in current main, but not on this branch |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3569 +/- ##
=======================================
Coverage 98.27% 98.28%
=======================================
Files 328 328
Lines 14824 14835 +11
Branches 2362 2362
=======================================
+ Hits 14569 14580 +11
Misses 116 116
Partials 139 139 ☔ View full report in Codecov by Sentry. |
|
Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/3569 |
* fix(warnings): Do not warn for default handlers * test: add test to check (lack of) warnings --------- Co-authored-by: Alc-Alc <alc@localhost> (cherry picked from commit 064c8fe)
Description
__init_subclass__
es for the default handlers.Closes
Closes #3552