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(ios): When collect-modules fails to find a node path exit without errors #4570

Merged
merged 4 commits into from
Feb 20, 2025

Conversation

krystofwoldrich
Copy link
Member

No description provided.

… errors

When the log has `error:` prefix xcode will exit with non-zero exit code.
Copy link
Contributor

github-actions bot commented Feb 19, 2025

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 392.58 ms 425.17 ms 32.59 ms
Size 17.75 MiB 20.11 MiB 2.37 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d997097 470.23 ms 475.46 ms 5.23 ms
3f05680 397.91 ms 405.65 ms 7.74 ms
148f924 492.65 ms 500.28 ms 7.63 ms
b6f8ea2 472.49 ms 469.60 ms -2.89 ms
13b68d9 424.22 ms 419.65 ms -4.57 ms
21a0abb 442.40 ms 422.88 ms -19.52 ms
e99226d 503.41 ms 491.61 ms -11.80 ms
690220d 468.28 ms 480.06 ms 11.78 ms
8de2810 430.47 ms 428.72 ms -1.75 ms
cdf2f33 469.46 ms 462.17 ms -7.29 ms

App size

Revision Plain With Sentry Diff
d997097 17.75 MiB 20.11 MiB 2.36 MiB
3f05680 17.75 MiB 20.11 MiB 2.37 MiB
148f924 17.73 MiB 19.94 MiB 2.21 MiB
b6f8ea2 17.75 MiB 20.11 MiB 2.36 MiB
13b68d9 17.75 MiB 20.11 MiB 2.37 MiB
21a0abb 17.75 MiB 20.11 MiB 2.36 MiB
e99226d 17.75 MiB 20.11 MiB 2.36 MiB
690220d 17.74 MiB 20.08 MiB 2.35 MiB
8de2810 17.74 MiB 20.08 MiB 2.34 MiB
cdf2f33 17.74 MiB 20.08 MiB 2.34 MiB

Previous results on branch: krystofwoldrich-patch-1

Startup times

Revision Plain With Sentry Diff
c5518c9 486.09 ms 481.46 ms -4.63 ms

App size

Revision Plain With Sentry Diff
c5518c9 17.75 MiB 20.11 MiB 2.37 MiB

Copy link
Contributor

github-actions bot commented Feb 19, 2025

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 409.86 ms 397.90 ms -11.96 ms
Size 7.15 MiB 8.38 MiB 1.23 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d0bf494+dirty 253.73 ms 308.23 ms 54.49 ms
fc150fe+dirty 409.55 ms 407.37 ms -2.18 ms
457e29f+dirty 591.49 ms 612.96 ms 21.47 ms
afe5fcb+dirty 396.84 ms 449.40 ms 52.56 ms
80b2ce3+dirty 271.29 ms 316.47 ms 45.18 ms
34aba08+dirty 331.79 ms 376.69 ms 44.91 ms
e5c9b8b+dirty 335.40 ms 360.06 ms 24.67 ms
9a3ca65+dirty 344.96 ms 358.92 ms 13.96 ms
ee4aab8+dirty 494.84 ms 574.57 ms 79.73 ms
488c9c5+dirty 448.98 ms 531.62 ms 82.64 ms

App size

Revision Plain With Sentry Diff
d0bf494+dirty 7.15 MiB 8.04 MiB 910.85 KiB
fc150fe+dirty 7.15 MiB 8.38 MiB 1.23 MiB
457e29f+dirty 7.15 MiB 8.10 MiB 981.29 KiB
afe5fcb+dirty 7.15 MiB 8.38 MiB 1.23 MiB
80b2ce3+dirty 7.15 MiB 8.04 MiB 911.02 KiB
34aba08+dirty 7.15 MiB 8.07 MiB 946.13 KiB
e5c9b8b+dirty 7.15 MiB 8.10 MiB 980.41 KiB
9a3ca65+dirty 7.15 MiB 8.09 MiB 962.83 KiB
ee4aab8+dirty 7.15 MiB 8.38 MiB 1.23 MiB
488c9c5+dirty 7.15 MiB 8.38 MiB 1.23 MiB

Previous results on branch: krystofwoldrich-patch-1

Startup times

Revision Plain With Sentry Diff
c5518c9+dirty 363.04 ms 351.51 ms -11.53 ms

App size

Revision Plain With Sentry Diff
c5518c9+dirty 7.15 MiB 8.38 MiB 1.23 MiB

Copy link
Contributor

github-actions bot commented Feb 19, 2025

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1220.27 ms 1208.18 ms -12.08 ms
Size 2.63 MiB 3.74 MiB 1.11 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
3261206+dirty 1219.49 ms 1224.10 ms 4.61 ms
1d86dd6+dirty 1249.71 ms 1279.16 ms 29.45 ms
8900e1a+dirty 1210.27 ms 1218.66 ms 8.39 ms
4cc5c27+dirty 1211.45 ms 1214.60 ms 3.16 ms
6e8584e+dirty 1274.50 ms 1296.82 ms 22.32 ms
e73f4ed+dirty 1243.27 ms 1244.52 ms 1.25 ms
6c56eb1+dirty 1224.29 ms 1232.00 ms 7.71 ms
3853f43+dirty 1221.82 ms 1242.64 ms 20.82 ms
a18210a+dirty 1237.15 ms 1224.85 ms -12.29 ms
cdf2f33+dirty 1227.71 ms 1233.94 ms 6.22 ms

App size

Revision Plain With Sentry Diff
3261206+dirty 2.63 MiB 3.69 MiB 1.06 MiB
1d86dd6+dirty 2.36 MiB 2.89 MiB 535.43 KiB
8900e1a+dirty 2.36 MiB 2.83 MiB 479.25 KiB
4cc5c27+dirty 2.36 MiB 3.04 MiB 698.52 KiB
6e8584e+dirty 2.36 MiB 2.88 MiB 533.17 KiB
e73f4ed+dirty 2.36 MiB 2.82 MiB 469.44 KiB
6c56eb1+dirty 2.36 MiB 3.11 MiB 759.87 KiB
3853f43+dirty 2.36 MiB 2.85 MiB 499.81 KiB
a18210a+dirty 2.63 MiB 3.70 MiB 1.06 MiB
cdf2f33+dirty 2.36 MiB 3.10 MiB 751.38 KiB

Previous results on branch: krystofwoldrich-patch-1

Startup times

Revision Plain With Sentry Diff
c5518c9+dirty 1230.16 ms 1238.59 ms 8.43 ms

App size

Revision Plain With Sentry Diff
c5518c9+dirty 2.63 MiB 3.74 MiB 1.11 MiB

Copy link
Contributor

github-actions bot commented Feb 19, 2025

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1225.13 ms 1219.69 ms -5.44 ms
Size 3.19 MiB 4.31 MiB 1.12 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
d16beca+dirty 1239.00 ms 1233.89 ms -5.11 ms
5a22220+dirty 1246.18 ms 1249.61 ms 3.43 ms
2534337+dirty 1220.87 ms 1221.47 ms 0.60 ms
1d86dd6+dirty 1289.25 ms 1293.36 ms 4.11 ms
ddc0552+dirty 1246.00 ms 1241.98 ms -4.02 ms
205982c+dirty 1228.06 ms 1237.72 ms 9.66 ms
e4d9fe0+dirty 1210.45 ms 1204.23 ms -6.21 ms
b1e8712+dirty 1284.11 ms 1297.82 ms 13.71 ms
afe5fcb+dirty 1223.53 ms 1223.90 ms 0.37 ms
21a0abb+dirty 1224.37 ms 1225.61 ms 1.24 ms

App size

Revision Plain With Sentry Diff
d16beca+dirty 2.92 MiB 3.69 MiB 790.48 KiB
5a22220+dirty 2.92 MiB 3.48 MiB 575.81 KiB
2534337+dirty 2.92 MiB 3.43 MiB 529.76 KiB
1d86dd6+dirty 2.92 MiB 3.44 MiB 538.27 KiB
ddc0552+dirty 2.92 MiB 3.67 MiB 772.47 KiB
205982c+dirty 2.92 MiB 3.67 MiB 772.63 KiB
e4d9fe0+dirty 3.19 MiB 4.26 MiB 1.08 MiB
b1e8712+dirty 2.92 MiB 3.40 MiB 494.15 KiB
afe5fcb+dirty 3.19 MiB 4.26 MiB 1.08 MiB
21a0abb+dirty 3.19 MiB 4.26 MiB 1.08 MiB

Previous results on branch: krystofwoldrich-patch-1

Startup times

Revision Plain With Sentry Diff
c5518c9+dirty 1217.02 ms 1216.92 ms -0.10 ms

App size

Revision Plain With Sentry Diff
c5518c9+dirty 3.19 MiB 4.31 MiB 1.12 MiB

Copy link
Collaborator

@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.

LGTM 🚀

@krystofwoldrich krystofwoldrich enabled auto-merge (squash) February 20, 2025 09:04
@krystofwoldrich krystofwoldrich merged commit d09472b into main Feb 20, 2025
63 checks passed
@krystofwoldrich krystofwoldrich deleted the krystofwoldrich-patch-1 branch February 20, 2025 09:06
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.

2 participants