-
Notifications
You must be signed in to change notification settings - Fork 522
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(litestar): Add failed_request_status_codes
#4021
feat(litestar): Add failed_request_status_codes
#4021
Conversation
failed_request_status_codes
Hey @vrslev, thank you for this PR! Code-wise looks great to me, I'll take it for a spin and prepare the docs for the change and then we should be ready to go. No action needed from your side, I'll ping you if that changes. Thanks again! |
@sentrivana that's great! I've prepared docs myself already :) |
Awesome! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4021 +/- ##
==========================================
+ Coverage 80.21% 80.23% +0.01%
==========================================
Files 139 139
Lines 15403 15411 +8
Branches 2601 2603 +2
==========================================
+ Hits 12356 12365 +9
- Misses 2203 2209 +6
+ Partials 844 837 -7
|
See #3134
Thank you for contributing to
sentry-python
! Please add tests to validate your changes, and lint your code usingtox -e linters
.Running the test suite on your PR might require maintainer approval. The AWS Lambda tests additionally require a maintainer to add a special label, and they will fail until this label is added.