-
-
Notifications
You must be signed in to change notification settings - Fork 341
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: Deserialize User #4732
feat: Deserialize User #4732
Conversation
|
Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
04e8394 | 1223.24 ms | 1244.33 ms | 21.08 ms |
8dae1f6 | 1212.60 ms | 1226.52 ms | 13.93 ms |
8a562cc | 1228.57 ms | 1245.18 ms | 16.60 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
04e8394 | 22.31 KiB | 771.93 KiB | 749.62 KiB |
8dae1f6 | 22.31 KiB | 772.51 KiB | 750.20 KiB |
8a562cc | 22.31 KiB | 773.33 KiB | 751.02 KiB |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feat/deserializing-events #4732 +/- ##
===============================================================
+ Coverage 91.082% 91.259% +0.176%
===============================================================
Files 627 627
Lines 72288 72795 +507
Branches 25675 26511 +836
===============================================================
+ Hits 65842 66432 +590
+ Misses 6351 6267 -84
- Partials 95 96 +1
... and 35 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
0ec0fd1
into
feat/deserializing-events
This PR is based on #4724.
Adds the Decodable implementation for SentryUser, including decoding arbitrary data.
#skip-changelog