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: tuist users not able to install CIO iOS SDK SPM manifest issue #791

Merged
merged 1 commit into from
Aug 8, 2024

Conversation

levibostian
Copy link
Contributor

Resolves: #790

Problem

iOS developers who want to use tools such as Tuist to generate xcode projects are currently experiencing an error when resolving the CIO iOS SDK via SPM.

See this PR for the full details of the problem and solution.

Solution

In order for customers to be able to install the iOS SDK via tuist, the iOS SDK's SPM manifest file must point to the correct version of CioAnalytics that includes the bug fix.

After this PR gets deployed, customers should be able to install the iOS SDK via tuist on the latest version.

Testing

Installed this git branch in SPM and Cocoapods sample apps. Verified that the correct source code was downloaded and that the apps compile successfully.

I am also going to run the sample apps quick that are built in this PR to verify that CDP API calls work as expected.

@levibostian levibostian requested a review from Shahroz16 August 8, 2024 15:03
Copy link

github-actions bot commented Aug 8, 2024

Sample app builds 📱

Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.


  • CocoaPods-FCM: Build failed. See CI job logs to determine the issue and try re-building.
  • APN-UIKit: levi/fix-tuist-spm (1723129470)
  • CocoaPods-FCM: levi/fix-tuist-spm (1723129823)

Copy link

codecov bot commented Aug 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.29%. Comparing base (2f4e4e4) to head (715379a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #791      +/-   ##
==========================================
+ Coverage   63.26%   63.29%   +0.02%     
==========================================
  Files         140      140              
  Lines        3999     3999              
==========================================
+ Hits         2530     2531       +1     
+ Misses       1469     1468       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@Shahroz16 Shahroz16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🎉

@levibostian levibostian merged commit cd1346a into main Aug 8, 2024
10 of 11 checks passed
@levibostian levibostian deleted the levi/fix-tuist-spm branch August 8, 2024 15:48
github-actions bot pushed a commit that referenced this pull request Aug 8, 2024
## [3.3.3](3.3.2...3.3.3) (2024-08-08)

### Bug Fixes

* tuist users not able to install CIO iOS SDK SPM manifest issue ([#791](#791)) ([cd1346a](cd1346a))
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.

CioAnalytics library have wrong sources format
2 participants