Skip to content

Conversation

@itaybre
Copy link
Contributor

@itaybre itaybre commented Nov 18, 2025

Test using Cirrus Runners images with xcode version set

@itaybre itaybre added the ready-to-merge Use this label to trigger all PR workflows label Nov 18, 2025
@github-actions
Copy link
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1230.61 ms 1263.06 ms 32.45 ms
Size 24.15 KiB 1.01 MiB 1014.90 KiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
01faa71 1238.81 ms 1263.98 ms 25.17 ms
7eafbde 1212.16 ms 1243.64 ms 31.48 ms
e8e40fd 1222.10 ms 1259.91 ms 37.80 ms
0ac4c65 1221.75 ms 1251.69 ms 29.94 ms
ea3f7b6 1228.14 ms 1258.71 ms 30.57 ms
80a5166 1224.49 ms 1251.29 ms 26.80 ms
939d583 1209.96 ms 1251.09 ms 41.13 ms
3c86ff3 1227.82 ms 1239.73 ms 11.91 ms
f8cad3c 1217.94 ms 1257.96 ms 40.02 ms
c174e5e 1209.65 ms 1238.73 ms 29.08 ms

App size

Revision Plain With Sentry Diff
01faa71 23.75 KiB 926.77 KiB 903.02 KiB
7eafbde 23.75 KiB 927.53 KiB 903.78 KiB
e8e40fd 23.75 KiB 1022.79 KiB 999.04 KiB
0ac4c65 23.75 KiB 968.16 KiB 944.41 KiB
ea3f7b6 23.75 KiB 1.01 MiB 1008.77 KiB
80a5166 23.75 KiB 904.53 KiB 880.78 KiB
939d583 23.75 KiB 1023.82 KiB 1000.07 KiB
3c86ff3 23.75 KiB 919.69 KiB 895.94 KiB
f8cad3c 23.75 KiB 1.01 MiB 1016.13 KiB
c174e5e 23.75 KiB 953.92 KiB 930.16 KiB

@itaybre
Copy link
Contributor Author

itaybre commented Nov 19, 2025

Confirmed by Cirrus Runners that these images are just syntactic sugar, so they actually contain all other images.

Because of this, there is no benefit on using them instead of the regular image.
Closing the PR

@itaybre itaybre closed this Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Use this label to trigger all PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants