Skip to content

Conversation

@jleandroperez
Copy link
Contributor

Details:

It's nearly 2018, and with Swift 5 around the corner... what a best time to flick the switch, and move over to Swift 4. Right?

Fixes #807

To test:

  • Verify that the unit tests are green
  • Run a smoke test over the editor


///
/// Returns the collection of Typing Attributes, with all of the available 'String' keys properly converted into
/// NSAttributedStringKey. Also known as: what you would expect from the SDK.
Copy link
Contributor

Choose a reason for hiding this comment

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

😆

@@ -0,0 +1,48 @@
import Foundation

Copy link
Contributor

Choose a reason for hiding this comment

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

Not really important, but remove 1 line here.

@jleandroperez
Copy link
Contributor Author

Let's do it! thanks for the review @diegoreymendez !!

@jleandroperez jleandroperez merged commit 8c11f68 into develop Nov 7, 2017
@jleandroperez jleandroperez deleted the issue/807-swift-4-migration branch November 7, 2017 20:18
@diegoreymendez diegoreymendez added this to the Version 1.0.0 Beta 15 milestone Nov 24, 2017
@diegoreymendez diegoreymendez changed the title Swift 4 Migration Migrates to Swift 4. Nov 24, 2017
@diegoreymendez diegoreymendez changed the title Migrates to Swift 4. Migrates the project to Swift 4. Nov 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants