Skip to content
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(test) null check for NDK internal metrics #2012

Merged
merged 1 commit into from
Apr 12, 2024

Conversation

YYChen01988
Copy link
Contributor

Goal

Extra null checks for NDK internal metrics to prevent a potential issue happens when next event is null.

Changeset

Added event null checks in NDK internal metrics class

Testing

Relied on existing tests

@YYChen01988 YYChen01988 requested a review from lemnik April 11, 2024 16:09
@bugsnagbot
Copy link
Collaborator

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1844.49 1668.58
arm64_v8a 626.95 450.82
armeabi_v7a 561.42 385.29
x86 704.75 524.53
x86_64 671.99 495.86

Generated by 🚫 Danger

Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@YYChen01988 YYChen01988 merged commit 61d3000 into next Apr 12, 2024
39 checks passed
@YYChen01988 YYChen01988 deleted the PLAT-11911/Crash_from_NDK_internal_metrics branch April 12, 2024 08:29
lemnik pushed a commit that referenced this pull request Jun 26, 2024
lemnik added a commit that referenced this pull request Jun 27, 2024
Backport: null check for NDK internal metrics (#2012)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants