sentry
sentry copied to clipboard
perf: move interfaces and integrations to orjson
trafficstars
Part of orjson move to get rid of sentry.utils.json
Codecov Report
Attention: Patch coverage is 82.92683% with 14 lines in your changes are missing coverage. Please review.
Project coverage is 77.88%. Comparing base (
dfc0962) to head (3e7377d).
:exclamation: Current head 3e7377d differs from pull request most recent head 6bf3625
Please upload reports for the commit 6bf3625 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## master #71055 +/- ##
===========================================
+ Coverage 41.97% 77.88% +35.90%
===========================================
Files 6519 6528 +9
Lines 290525 290851 +326
Branches 50278 50339 +61
===========================================
+ Hits 121959 226519 +104560
+ Misses 161830 58085 -103745
+ Partials 6736 6247 -489
| Files | Coverage Δ | |
|---|---|---|
| src/sentry/interfaces/security.py | 89.36% <100.00%> (+19.14%) |
:arrow_up: |
| src/sentry/shared_integrations/client/base.py | 87.45% <100.00%> (+44.93%) |
:arrow_up: |
| .../sentry/shared_integrations/exceptions/__init__.py | 91.89% <100.00%> (+27.92%) |
:arrow_up: |
| src/sentry/shared_integrations/response/base.py | 71.18% <100.00%> (+28.81%) |
:arrow_up: |
| src/sentry/testutils/helpers/backups.py | 99.41% <100.00%> (+2.63%) |
:arrow_up: |
| ...y/testutils/performance_issues/event_generators.py | 94.59% <100.00%> (+24.32%) |
:arrow_up: |
| src/sentry/utils/assets.py | 100.00% <100.00%> (+20.83%) |
:arrow_up: |
| src/sentry/utils/codecs.py | 100.00% <100.00%> (+28.88%) |
:arrow_up: |
| src/sentry/utils/email/message_builder.py | 91.89% <100.00%> (+48.35%) |
:arrow_up: |
| src/sentry/utils/outcomes.py | 98.11% <100.00%> (+9.65%) |
:arrow_up: |
| ... and 8 more |
PR reverted: 191f282684c8ce4f10c82b3232041a1f7934de8c