Skip to content

Conversation

@thedavidharris
Copy link
Contributor

Seems to fix #350. Can verify by just doing an Xcode archive on this. Did not see any need to make changes to the tvOS or macOS targets, those seems to remain okay.

@stephencelis
Copy link
Member

@thedavidharris Unfortunately I'm still getting those same errors when I run pod lib lint on your branch ☹️

@thedavidharris
Copy link
Contributor Author

@stephencelis forgot to update the Podspec. Not on a machine I can validate it, but I think that would be why pod lib lint didn't pick it up.

@thedavidharris
Copy link
Contributor Author

Yeah, just checked, this does fix pod lib lint too. It's a weird issue, although given the low use of iOS 10 and the fact that this is a test library, should be fine to bump it here.

zweigraf added a commit to SwiftRex/TestingExtensions that referenced this pull request Jun 18, 2020
Using https://github.com/thedavidharris/swift-snapshot-testing.git, .revision(64c1925) for now, as the current release of SnapshotTesting does not compile.

See this issue for more details: pointfreeco/swift-snapshot-testing#350

 We are waiting for this PR to be merged, then we can revert this change: pointfreeco/swift-snapshot-testing#355
Copy link
Member

@stephencelis stephencelis left a comment

Choose a reason for hiding this comment

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

Looks great, thanks! Surprised tvOS didn't need to be bumped...

@stephencelis stephencelis merged commit 2fd9135 into pointfreeco:master Jun 19, 2020
@thedavidharris thedavidharris deleted the master branch September 26, 2022 02:29
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.

CocoaPods fails to validate SnapshotTesting 1.8.0+

2 participants