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

Xcode 10 and Swift 4.2 fix #105

Merged
merged 4 commits into from
Oct 21, 2018
Merged

Xcode 10 and Swift 4.2 fix #105

merged 4 commits into from
Oct 21, 2018

Conversation

yasinkavakliat
Copy link
Contributor

renamed kCALineJoinBevel to CAShapeLayerLineJoin.bevel so that SwiftChart is able to compile inside Xcode 10 and project set to Swift 4.2

This is my first PR on an open source project so if there is something false, please let me know 🙌

@dstranz
Copy link
Contributor

dstranz commented Oct 2, 2018

@Coupz You could also add s.swift_version = '4.2' to SwiftChart.podspec file and update .swift-version file to 4.2.

@yasinkavakliat
Copy link
Contributor Author

Made the changes that @dstranz wrote above.

@dstranz
Copy link
Contributor

dstranz commented Oct 4, 2018

Nice. Now we need to wait for @gpbl to accept this PR.

@yasinkavakliat
Copy link
Contributor Author

yasinkavakliat commented Oct 4, 2018

@dstranz has updated the example project. Thanks for that 👍
Is this project alive @gpbl ?

BTW @dstranz thank you for your help 👍

@gpbl
Copy link
Owner

gpbl commented Oct 21, 2018

Yes it is alive again :) Sorry folks for the delays and thanks for the PR.

@gpbl gpbl merged commit 4949a94 into gpbl:master Oct 21, 2018
@pushchris
Copy link

@gpbl could we get a new release pushed out so that folks using pods can update? Thanks!

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