Skip to content

ref: Migrate sample apps to package references#8109

Draft
NinjaLikesCheez wants to merge 19 commits into
mainfrom
migrate-sample-apps-to-package-references
Draft

ref: Migrate sample apps to package references#8109
NinjaLikesCheez wants to merge 19 commits into
mainfrom
migrate-sample-apps-to-package-references

Conversation

@NinjaLikesCheez

Copy link
Copy Markdown
Member

📜 Description

Migrate Sample apps to use package references (instead of project references) so that we can enable traits when testing v10 & any future trait we introduce.

💡 Motivation and Context

Sample apps currently reference the Xcode project rather than the SPM package, which means that we lose SPM traits in the sample apps. This can be worked around by manually setting compiler flags. However, it would be nicer if the sample apps use package references instead and inherit the ability to set traits within the Xcode UI.

💚 How did you test it?

This is a WIP - this section will be updated as time flows inevitably past us all. For now, I'm using 'it builds' as the check to move onto the next sample app and before setting this to 'ready-to-merge' I'll be doing a more thorough testing (including CI)

📝 Checklist

You have to check all boxes before merging:

  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • If I added a new public API, I also added it to the SentryObjC wrapper.

#skip-changelog

@NinjaLikesCheez NinjaLikesCheez changed the title Migrate sample apps to package references ref: Migrate sample apps to package references Jun 17, 2026
@NinjaLikesCheez NinjaLikesCheez force-pushed the migrate-sample-apps-to-package-references branch from 83ef117 to 57ad17c Compare June 18, 2026 09:51
@NinjaLikesCheez NinjaLikesCheez added the ready-to-merge Use this label to trigger all PR workflows label Jun 18, 2026
@NinjaLikesCheez

Copy link
Copy Markdown
Member Author

Added label to trigger the CI, will move out of draft when it's ready for review

@sentry

sentry Bot commented Jun 18, 2026

Copy link
Copy Markdown

📲 Install Builds

iOS

🔗 App Name App ID Version Configuration
SDK-Size io.sentry.sample.SDK-Size 9.18.0 (1) Release

⚙️ sentry-cocoa Build Distribution Settings

@github-actions

github-actions Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1231.71 ms 1262.72 ms 31.01 ms
Size 24.14 KiB 1.19 MiB 1.16 MiB

Baseline results on branch: main

Startup times

Revision Plain With Sentry Diff
bdd8e0e 1233.35 ms 1266.96 ms 33.60 ms
92fada5 1217.60 ms 1252.89 ms 35.30 ms
fc4e913 1231.79 ms 1267.89 ms 36.11 ms
e56c394 1229.37 ms 1245.86 ms 16.49 ms
39655d1 1224.05 ms 1258.38 ms 34.33 ms
ccf353a 1233.79 ms 1268.77 ms 34.98 ms
bd2b6d3 1213.78 ms 1253.83 ms 40.06 ms
8693a8f 1222.37 ms 1249.31 ms 26.95 ms
ae8cece 1216.83 ms 1251.37 ms 34.55 ms
ce1ec47 1212.60 ms 1247.61 ms 35.02 ms

App size

Revision Plain With Sentry Diff
bdd8e0e 24.14 KiB 1.16 MiB 1.13 MiB
92fada5 24.14 KiB 1.17 MiB 1.15 MiB
fc4e913 24.14 KiB 1.17 MiB 1.15 MiB
e56c394 24.14 KiB 1.16 MiB 1.13 MiB
39655d1 24.14 KiB 1.16 MiB 1.14 MiB
ccf353a 24.14 KiB 1.17 MiB 1.15 MiB
bd2b6d3 24.14 KiB 1.17 MiB 1.14 MiB
8693a8f 24.14 KiB 1.16 MiB 1.13 MiB
ae8cece 24.14 KiB 1.15 MiB 1.13 MiB
ce1ec47 24.14 KiB 1.15 MiB 1.13 MiB

Previous results on branch: migrate-sample-apps-to-package-references

Startup times

Revision Plain With Sentry Diff
fba90a8 1221.17 ms 1253.23 ms 32.07 ms
435caa9 1211.65 ms 1248.80 ms 37.16 ms

App size

Revision Plain With Sentry Diff
fba90a8 24.14 KiB 1.18 MiB 1.16 MiB
435caa9 24.14 KiB 1.18 MiB 1.16 MiB

@NinjaLikesCheez NinjaLikesCheez self-assigned this Jun 19, 2026
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.

Migrate Sample Apps to use Package References instead of Project References

1 participant