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

Swift 3.0 #45

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Swift 3.0 #45

wants to merge 14 commits into from

Conversation

thekie
Copy link

@thekie thekie commented Sep 27, 2016

Hello,
I updated the code to compile with xcode8 and swift3. I also needed to update the screenshorts for the visual tests, because the rendering on iOS 10 is some pixel different.

Cheers

Updated FBSnapshotTestCase to version 2.1.2 for swift3
Updated podfile syntax
	* xcpretty uses sub gem wich needs at least 2.2.2
	* pod file to enable swift3 needs at least cocoapods 1.0
Updated screenshots for tests again, because testing with xcjobs created pixel differences again. (Maybe that was the differences i found while testing with Xcode and tried to correct in commit #5026d7a)
…es again so the screenshots are updated also.
@thekie
Copy link
Author

thekie commented Sep 27, 2016

I am not able to get the signing running for the CI, but the tests are running on my machine locally. @watsonbox perhaps you could merge this and check the signing with your profile like it was before. thanks.

@Treverr
Copy link

Treverr commented Oct 28, 2016

Do we know if this development is going to be kept up to date? Or is there a way to import this into my project through CocoaPods

@thekie
Copy link
Author

thekie commented Oct 29, 2016

Hey @Treverr ,
I dont know if this is still maintained. But you can use my version of ios_google_places_autocomplete, by adding
pod 'GooglePlacesAutocomplete', :git => 'https://github.com/danielkiedrowski/ios_google_places_autocomplete.git'
to you Podfile.

@thekie
Copy link
Author

thekie commented Dec 8, 2016

@watsonbox Is this still maintained? I'd love to help!

@sishpal
Copy link

sishpal commented Jan 25, 2017

example not woking properly

@thekie
Copy link
Author

thekie commented Dec 12, 2017

@watsonbox @sishpal How about now?

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.

3 participants