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 #422

Merged
merged 5 commits into from
Jun 4, 2017
Merged

Swift 4 #422

merged 5 commits into from
Jun 4, 2017

Conversation

mdiep
Copy link
Contributor

@mdiep mdiep commented Jun 1, 2017

This gets master building with the Swift 4 snapshots.

Changes fall into 2 categories:

  • Type inference failures
  • Removal of implicit tuple destructuring from SE-0110

- before_install:
- eval "$(curl -sL https://gist.githubusercontent.com/kylef/5c0475ff02b7c7671d2a/raw/9f442512a46d7a2af7b850d65a7e9bd31edfb09b/swiftenv-install.sh)"
script:
- swift build
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nimble doesn't build with Swift 4, so this only builds for now. 😞

@andersio
Copy link
Member

andersio commented Jun 1, 2017

Could you add a job for the Swift 3 compatibility mode?

@andersio
Copy link
Member

andersio commented Jun 1, 2017

(Unrelated to the PR) The lost of implicit 1-tuple destructing makes me wonder if we need to introduce sth like combineLatestMap or zipMap. 🤦‍♂️

@mdiep
Copy link
Contributor Author

mdiep commented Jun 2, 2017

Could you add a job for the Swift 3 compatibility mode?

I'm not sure how to do that, so I'd love to leave it for a separate PR.

@mdiep mdiep added this to the 2.0 milestone Jun 2, 2017
@andersio andersio merged commit c70e5cd into master Jun 4, 2017
@andersio andersio deleted the swift-4 branch June 4, 2017 15:14
interrupted: log(.interrupted),
terminated: log(.terminated),
disposed: log(.disposed),
completed: log(.completed) as ((()) -> Void)?,
Copy link
Member

Choose a reason for hiding this comment

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

Ugh

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.

None yet

3 participants