10000 Add cocoapods support for FactoryKit by tahirmt · Pull Request #304 · hmlongco/Factory · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add cocoapods support for FactoryKit #304

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

Closed
wants to merge 1 commit into from

Conversation

tahirmt
Copy link
Contributor
@tahirmt tahirmt commented May 20, 2025

Creating a new FactoryKit.podspec file to match the targets in SPM. The reason to do this is if there is code that supports both cocoapods and SPM and wants to migrate to the new FactoryKit target, they cannot do that without adding conditional import code for cocoapods.

@hmlongco
Copy link
Owner
hmlongco commented May 20, 2025

"The reason to do this is if there is code that supports both cocoapods and SPM..."

And how many do this?

As the CocoaPods support page points out, SPM is 9 years old now, and CP is on life-support only, to be frozen in the very near future.

@tahirmt
Copy link
Contributor Author
tahirmt commented May 20, 2025

That is very true. I suspect not a lot of library developers would depend on Factory.

@tahirmt
Copy link
Contributor Author
tahirmt commented May 21, 2025

There's already a FactoryKit on trunk anyway so this would never have published. I'll close the PR. But if someone needs it they can use private pod specs repo and use the podspec file to do it https://cocoapods.org/pods/FactoryKit

@tahirmt tahirmt closed this May 21, 2025
@tahirmt tahirmt d 80CD eleted the factorykit-cocoapods branch May 21, 2025 13:03
@hmlongco
Copy link
Owner

Didn't think to look to see if FactoryKit was available. Thanks for the help, regardless.

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.

2 participants
0