Skip to content

fix: Correct mixed language issue for spm#24

Merged
BrandonStalnaker merged 3 commits into
developmentfrom
fix/Correct-Mixed-Language-Issue-For-SPM
Aug 4, 2025
Merged

fix: Correct mixed language issue for spm#24
BrandonStalnaker merged 3 commits into
developmentfrom
fix/Correct-Mixed-Language-Issue-For-SPM

Conversation

@BrandonStalnaker

@BrandonStalnaker BrandonStalnaker commented Jul 31, 2025

Copy link
Copy Markdown
Collaborator

Summary

  • SPM could not integrate our latest version because we included Swift code along with the Objective-C code. This was not an issue when integrating with cocoaPods. To solve this problem we either needed to split the code into two language dependent targets or ship the framework as an xcframework binary. Since it was simpler and using binaries causes the SDK to be. harder to test I went with the first option

Testing Plan

  • Was this tested locally? If not, explain why.
  • Confirmed all unit tests and that the project file, package.swift file, and pod lib lint all run successfully. I also succesfully integrated the kit into swift and objective-c projects using SPM and cocoapods respectively.

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

@BrandonStalnaker BrandonStalnaker self-assigned this Jul 31, 2025
@BrandonStalnaker BrandonStalnaker requested a review from a team as a code owner July 31, 2025 15:53
@BrandonStalnaker BrandonStalnaker merged commit 61ce4a0 into development Aug 4, 2025
5 checks passed
@BrandonStalnaker BrandonStalnaker deleted the fix/Correct-Mixed-Language-Issue-For-SPM branch August 4, 2025 13:36
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