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

Restructure Demo - Use CocoaPods #430

Closed
wants to merge 5 commits into from
Closed

Conversation

scannillo
Copy link
Contributor

@scannillo scannillo commented Oct 19, 2023

Summary

We want to start converting the DropIn source code file-by-file from Objective-C to Swift. Starting this work revealed a limitation of the structure of our Demo app and local xcodeproject.

See error:
Screenshot 2023-10-19 at 11 28 57 AM

Currently our Demo uses SPM to pull in a local version of the DropIn source code. It is a known limitation that SPM does not support mixed language dependencies. So, with this setup we will not be able to run our Demo app as we convert file-by-file to Swift.

Changes

This PR restructures our Demo app to pull in a local version of DropIn via CocoaPods, instead of SPM.

  • Added a Podfile for the Demo app
  • Removed SPM frameworks from Demo xcodeproj

Checklist

  • Added a changelog entry

Authors

@scannillo

@scannillo scannillo requested a review from a team as a code owner October 19, 2023 17:47
@scannillo scannillo marked this pull request as draft October 19, 2023 17:47
@scannillo
Copy link
Contributor Author

I am messing around with how development with this structure will feel, so far the ObjC <--> Swift interop is tricky. We might prefer a "Carthage" style setup for our demo instead.

@scannillo
Copy link
Contributor Author

Closing in favor of using a Carthage-style setup for our project structure

@scannillo scannillo closed this Nov 20, 2023
@jaxdesmarais jaxdesmarais deleted the restructure-demo-cocoapods branch August 1, 2024 18:46
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.

1 participant