-
Notifications
You must be signed in to change notification settings - Fork 332
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
Swiftify interface #103
Swiftify interface #103
Conversation
Oh awesome! I tried to do this myself in the past and got stuck due to not know about about Can you verify that these changes don't break the # XMPPFramework with Swift extensions
pod 'XMPPFramework/Swift', :git => 'https://github.com/robbiehanson/XMPPFramework.git', :branch => 'master'
pod 'KissXML', :git => 'https://github.com/MacPass/KissXML.git', :branch => 'swiftless' |
I just build XMPPFramework with the proposed KissXML version and the build succeeds. I have no idea if any runtime stuff is broken though. |
Both macOS and iOS? |
Yup. Both builds succeeded. I didn't see any unit test, are there any to ensure the functionality is actually correct? |
There are unit tests in the |
This is an attempt to remove the need of a dedicated Swift source file to make KissXML pure Obj-C again. Please consider this PR with caution as I am no Swift expert. The tests for iOS run fine, macOS test did crash in NSXml under Xcode 9.2 and macOS 10.12 for unknown reasons.