-
Notifications
You must be signed in to change notification settings - Fork 381
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
DRY transport exceptions #4484
base: master
Are you sure you want to change the base?
DRY transport exceptions #4484
Conversation
Thank you for updating Change log entry section 👏 Visited at: 2025-03-12 14:14:55 UTC |
Datadog ReportBranch report: ✅ 0 Failed, 20626 Passed, 1373 Skipped, 3m 16.34s Total Time |
3b6eb54
to
b2120ae
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #4484 +/- ##
=======================================
Coverage 97.70% 97.70%
=======================================
Files 1379 1381 +2
Lines 83881 83903 +22
Branches 4251 4256 +5
=======================================
+ Hits 81953 81980 +27
+ Misses 1928 1923 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
What does this PR do?
Moves the "endpoint not supported" and "endpoint not defined" exceptions into the core.
Motivation:
Currently these exceptions are defined for each endpoint but their contents is identical.
Change log entry
None
Additional Notes:
How to test the change?
Existing unit tests