Skip to content

Fix: Fix compile error in v7#5134

Merged
antonis merged 4 commits into
release/7.0.1from
fixCompileError
Sep 3, 2025
Merged

Fix: Fix compile error in v7#5134
antonis merged 4 commits into
release/7.0.1from
fixCompileError

Conversation

@noahsmartin
Copy link
Copy Markdown
Contributor

@noahsmartin noahsmartin commented Sep 2, 2025

The latest v7 release is causing CI to break in sentry-cocoa, this should fix it

#skip-changelog

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 2, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1217.98 ms 1221.88 ms 3.90 ms
Size 3.19 MiB 4.44 MiB 1.25 MiB

Previous results on branch: fixCompileError

Startup times

Revision Plain With Sentry Diff
41c41b3+dirty 1213.76 ms 1209.60 ms -4.16 ms
b471ffe+dirty 1233.81 ms 1235.98 ms 2.17 ms

App size

Revision Plain With Sentry Diff
41c41b3+dirty 3.19 MiB 4.44 MiB 1.25 MiB
b471ffe+dirty 3.19 MiB 4.44 MiB 1.25 MiB

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 2, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1228.31 ms 1222.60 ms -5.71 ms
Size 2.63 MiB 3.87 MiB 1.24 MiB

Previous results on branch: fixCompileError

Startup times

Revision Plain With Sentry Diff
41c41b3+dirty 1234.92 ms 1237.92 ms 3.00 ms
b471ffe+dirty 1209.71 ms 1216.72 ms 7.01 ms

App size

Revision Plain With Sentry Diff
41c41b3+dirty 2.63 MiB 3.87 MiB 1.24 MiB
b471ffe+dirty 2.63 MiB 3.87 MiB 1.24 MiB

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 2, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 459.49 ms 483.33 ms 23.84 ms
Size 17.75 MiB 19.68 MiB 1.93 MiB

Previous results on branch: fixCompileError

Startup times

Revision Plain With Sentry Diff
b471ffe 360.53 ms 420.00 ms 59.47 ms
41c41b3 417.12 ms 423.94 ms 6.82 ms

App size

Revision Plain With Sentry Diff
b471ffe 17.75 MiB 19.68 MiB 1.93 MiB
41c41b3 17.75 MiB 19.68 MiB 1.93 MiB

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Sep 2, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 373.65 ms 411.88 ms 38.23 ms
Size 7.15 MiB 8.41 MiB 1.26 MiB

Previous results on branch: fixCompileError

Startup times

Revision Plain With Sentry Diff
b471ffe+dirty 338.81 ms 414.98 ms 76.17 ms
41c41b3+dirty 391.20 ms 437.52 ms 46.32 ms

App size

Revision Plain With Sentry Diff
b471ffe+dirty 7.15 MiB 8.41 MiB 1.26 MiB
41c41b3+dirty 7.15 MiB 8.41 MiB 1.26 MiB

Copy link
Copy Markdown
Contributor

@antonis antonis left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this @noahsmartin 🙇
LGTM 🚀

Comment thread packages/core/ios/RNSentry.mm Outdated
lucas-zimerman and others added 2 commits September 3, 2025 12:48
Co-authored-by: Antonis Lilis <antonis.lilis@gmail.com>
Copy link
Copy Markdown
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

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

LGTM!

@antonis antonis changed the base branch from main to release/7.0.1 September 3, 2025 12:07
Comment thread packages/core/ios/RNSentry.mm
@antonis antonis merged commit a7de1bc into release/7.0.1 Sep 3, 2025
61 checks passed
@antonis antonis deleted the fixCompileError branch September 3, 2025 12:13
@antonis
Copy link
Copy Markdown
Contributor

antonis commented Sep 3, 2025

Thank you for fixing this @noahsmartin. This is now release with https://github.com/getsentry/sentry-react-native/releases/tag/7.0.1

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