-
Notifications
You must be signed in to change notification settings - Fork 55
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
Carthage ios setup #46
Conversation
Hey @jessearmand, thanks for the issue! Can we get a bit more information about what is failing in Carthage? The snapshot testing dependency should only come into play when you want to include the If you are importing only And as for the iOS 10.0 bump, that makes sense to me! Only one thing to note, we use xcodegen to keep our project file in good shape for Carthage, so your changes will need to be made to |
@jessearmand I just merged #47, so if you merge master you can make the changes to Thanks! |
@mbrandonw Okay, I'm re-generating the project with xcodegen. I didn't know it exists. I didn't use html snapshot tests as a dependency, I didn't import any of it. My project bootstrap is just this simple:
First I've noticed this problem because it builds the
|
Hey @jessearmand! It's been awhile since I used Carthage, but any test target (including Because of this kinda thing we have a callout to this on the SnapshotTesting README that I think applies here: https://github.com/pointfreeco/swift-snapshot-testing#carthage So maybe we need to add a similar callout to the README here. |
Some configurations needed to successfully built with Carthage. The main issue is because
HtmlSnapshotTesting_iOS
has to be built together with building theHtml_iOS
SnapshotTesting
minimum deployment target is also 10.0SnapshotTesting
as private dependencySnapshotTesting
has bitcode disabled.