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.0 support added #421

Merged
merged 6 commits into from
Jan 24, 2018
Merged

Swift 4.0 support added #421

merged 6 commits into from
Jan 24, 2018

Conversation

lanserxt
Copy link
Contributor

No description provided.

@dcharbonnier
Copy link
Contributor

thx, can you fix the the travis configuration or the tests please ?

@lanserxt
Copy link
Contributor Author

Hello,

Looks like need to add

xcode_sdk: iphonesimulator10.0

Where can I change the .yml?

@dcharbonnier
Copy link
Contributor

dcharbonnier commented Jan 18, 2018 via email

Destination changed to platform=iOS Simulator,name=iPhone SE,OS=10.3.1
@gringoireDM
Copy link

On the swift 4 you might want to update travis config to use xCode 9.

Changed to osx_image: xcode9
@lanserxt
Copy link
Contributor Author

Will update tests also

@dcharbonnier
Copy link
Contributor

great, thanks @lanserxt !

@lanserxt
Copy link
Contributor Author

@dcharbonnier all done, safe to merge

Copy link

@gringoireDM gringoireDM left a comment

Choose a reason for hiding this comment

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

I would change version number and podspec to release a new pod version. Also the .swift-version file must be added to allow a new cocoa pods deploy.

See example for swift-version file.

@lanserxt
Copy link
Contributor Author

@gringoireDM What version should I set? Currently set 1.1 for Xcode and 0.10.2 for pod spec.
Swift version file added.

@@ -1,7 +1,7 @@
Pod::Spec.new do |s|
s.name = 'HanekeSwift'
s.module_name = 'Haneke'
s.version = '0.10.1'
s.version = '0.10.2'
Copy link

@gringoireDM gringoireDM Jan 22, 2018

Choose a reason for hiding this comment

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

The podspec version should be the same of the project version. the issue here is that pod support has been discontinued after 0.10.2. If you wanted latest commits you had to refer to specific branches or straight to master. this is not a best practice and a new pod version should be released. Good 1.1 as project version, but here too it should change to 1.1

@dcharbonnier: Please tag master after merge and release a new pod version. This is something we cannot do, so we need you to do it.

@dcharbonnier
Copy link
Contributor

No one has access to the pod account. If you want to release it you will have to change the name

@gringoireDM
Copy link

@dcharbonnier please do. Haneke_Swift maybe? To release new pod version is important if you want still maintain the project.

@dcharbonnier
Copy link
Contributor

Thx !!
For the pod, I don't even have a Mac anymore ;-)

@dcharbonnier dcharbonnier merged commit 500aece into Haneke:master Jan 24, 2018
@gringoireDM
Copy link

Just admit that the library is discontinued. Anyway we moved away from it. Thank you anyway.

@lanserxt
Copy link
Contributor Author

@dcharbonnier You are welcome!

@dcharbonnier
Copy link
Contributor

dcharbonnier commented Jan 24, 2018

With people like @lanserxt I think we can say it's alive and the more people will contribute the more this project will be alive.
If there is a better project, with more contributors or a project you like more, people can move away and are welcome back when they want :-) It's like cocoapod vs carthage.
Now if someone want to take the responsibility of deploying and maintaining a pod, he is free to do it and will get my support, this project is open.

@gringoireDM
Copy link

You don't even have a Mac to test the code you are going to approve. How can you say this is a maintained project?

I do understand the importance of contributions, but a pod cannot stand just on contributions. It's important that the maintainer helps as well at least with the most basic things like tagging releases, testing before merging a PR, releasing new pod versions. There are things that contributors cannot do. If you don't whish to do them either, handover the ownership of the repository or declare it unmaintained. In any case, like I said before, we already removed haneke as dependency since you are willing to approve anything that pass the travis build.

We are off topic.

@smhk
Copy link

smhk commented Jan 24, 2018

Hoy caramba !

Cocoapods is horrible. Carthage is the only way to go.

one vote!

(Soon, Swift Package Manager will be here and cocoapods will finally be dead.)

@Haneke Haneke locked as off-topic and limited conversation to collaborators Jan 24, 2018
@dcharbonnier
Copy link
Contributor

Join the chat at https://gitter.im/Haneke/HanekeSwift

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants