Skip to content

Reduce the level of warnings in the ExitInfo plugin#2221

Merged
lemnik merged 1 commit intonextfrom
lemnik/reduce-exitinfo-warnings
Jul 29, 2025
Merged

Reduce the level of warnings in the ExitInfo plugin#2221
lemnik merged 1 commit intonextfrom
lemnik/reduce-exitinfo-warnings

Conversation

@lemnik
Copy link
Copy Markdown
Contributor

@lemnik lemnik commented Jul 23, 2025

Goal

Avoid emitting potentially confusing warnings during normal exit-info plugin operations

@lemnik lemnik force-pushed the lemnik/reduce-exitinfo-warnings branch from c43a5e8 to 0ef7c37 Compare July 23, 2025 07:23
@bugsnagbot
Copy link
Copy Markdown
Collaborator

bugsnagbot commented Jul 23, 2025

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1828.02 1630.08
arm64_v8a 639.23 438.53
armeabi_v7a 581.9 381.19
x86 708.85 512.24
x86_64 684.28 483.58

Generated by 🚫 Danger

@lemnik lemnik force-pushed the lemnik/reduce-exitinfo-warnings branch from 0ef7c37 to 93fc319 Compare July 23, 2025 07:43
YYChen01988
YYChen01988 previously approved these changes Jul 28, 2025
Copy link
Copy Markdown
Contributor

@YYChen01988 YYChen01988 left a comment

Choose a reason for hiding this comment

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

LGTM

@lemnik lemnik force-pushed the lemnik/reduce-exitinfo-warnings branch from 93fc319 to 8a689e1 Compare July 29, 2025 08:04
Copy link
Copy Markdown
Contributor

@tomlongridge tomlongridge left a comment

Choose a reason for hiding this comment

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

Not sure it justifies a minor release on it's own, but we'll argue that out on release.

@lemnik lemnik requested a review from YYChen01988 July 29, 2025 08:28
@lemnik lemnik merged commit b08fcad into next Jul 29, 2025
35 checks passed
@lemnik lemnik deleted the lemnik/reduce-exitinfo-warnings branch July 29, 2025 08:29
This was referenced Jul 31, 2025
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.

4 participants