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 4 Support #704

Merged
merged 23 commits into from
Sep 14, 2017
Merged

Swift 4 Support #704

merged 23 commits into from
Sep 14, 2017

Conversation

onevcat
Copy link
Owner

@onevcat onevcat commented Jun 8, 2017

No description provided.

@lfarah
Copy link

lfarah commented Jul 18, 2017

@onevcat when are you merging this? Would love to continue using Kingfisher in my project!
screen shot 2017-07-18 at 11 46 03 am

@onevcat
Copy link
Owner Author

onevcat commented Jul 19, 2017

@lfarah It will not be merged until Swift 4 officially released. Before that, you could just use the "swift4" branch to continue use Kingfisher is your Swift 4 project.

@onevcat onevcat mentioned this pull request Jul 25, 2017
3 tasks
@lightsprint09
Copy link

Hey. There is an issues withe carthage and the new build system. Would it be ok to revert to the old build system??

@onevcat
Copy link
Owner Author

onevcat commented Jul 31, 2017

@lightsprint09
Fallback to legacy build system/

onevcat and others added 8 commits August 1, 2017 09:52
Disable gather coverage data in scheme test action
# Conflicts:
#	Kingfisher.xcodeproj/xcshareddata/xcschemes/Kingfisher-macOS.xcscheme
#	Kingfisher.xcodeproj/xcshareddata/xcschemes/Kingfisher-tvOS.xcscheme
#	Kingfisher.xcodeproj/xcshareddata/xcschemes/Kingfisher.xcscheme
#	Sources/ImageView+Kingfisher.swift
@onevcat onevcat merged commit 635dbf4 into master Sep 14, 2017
@onevcat onevcat deleted the swift4 branch September 16, 2017 12:59
petarlazarov pushed a commit to dowjones-mobile/Kingfisher that referenced this pull request Oct 6, 2021
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.

4 participants