chore: clean up CocoaPods podspec source globs#724
Conversation
- Limit mParticle/AppExtension source_files to .h and .m (no .mm/.cpp/.swift under tree) - Drop mParticleNoLocation subspec; SDK is single variant without Location Made-with: Cursor
Kit sources use only .h/.m; align CocoaPods patterns with the tree. Made-with: Cursor
PR SummaryMedium Risk Overview Kit podspec tightening: Updates multiple kit podspecs to drop Reviewed by Cursor Bugbot for commit 09b9c54. Bugbot is set up for automated code reviews on this repo. Configure here. |
📦 SDK Size Impact ReportMeasures how much the SDK adds to an app's size (with-SDK minus without-SDK).
➡️ SDK size impact change is minimal. Raw measurementsTarget branch (main): {"baseline_app_size_kb":84,"baseline_executable_size_bytes":75464,"with_sdk_app_size_kb":1880,"with_sdk_executable_size_bytes":76312,"sdk_impact_kb":1796,"sdk_executable_impact_bytes":848,"xcframework_size_kb":6532}This PR: {"baseline_app_size_kb":84,"baseline_executable_size_bytes":75464,"with_sdk_app_size_kb":1880,"with_sdk_executable_size_bytes":76312,"sdk_impact_kb":1796,"sdk_executable_impact_bytes":848,"xcframework_size_kb":6532} |
Background
CocoaPods globs listed extensions that are not present in the repo; the main SDK also exposed a redundant Location-disabled subspec now that the library ships a single variant.
What Has Changed
source_fileslimited to*.h/*.m; removedmParticleNoLocationsubspec.mmfrom*.{h,m,mm}patterns (kits have no Objective-C++ sources).Breaking change
Pods or apps that depend on
mParticle-Apple-SDK/mParticleNoLocationmust use the defaultmParticlesubspec ormParticle-Apple-SDKwithout that path (e.g. update Media SDK / Iterable kit podspecs in lockstep).Screenshots/Video
N/A