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

Merge Swift 3.1 branch into master #862

Closed
wants to merge 9 commits into from
Closed

Conversation

swizzlr
Copy link
Contributor

@swizzlr swizzlr commented Feb 2, 2017

Bring in Swift-3.1 fixes.

@swizzlr
Copy link
Contributor Author

swizzlr commented Feb 2, 2017

#843

@parkera
Copy link
Member

parkera commented Feb 2, 2017

This already has conflicts.

In general what we've been doing is this: putting changes in master, then cherry-picking to swift-3.1-branch. It's the process the other projects are using.

Personally I prefer to target fixes at the branch and merge to master, but that's not the current setup. I think we could revisit that next time we branch for a release.

@swizzlr
Copy link
Contributor Author

swizzlr commented Feb 2, 2017

Ah - ok. Is this documented anywhere or shall I add to README?

@swizzlr swizzlr closed this Feb 2, 2017
@parkera
Copy link
Member

parkera commented Feb 2, 2017

Mostly it's just emails to the various dev lists.

@parkera
Copy link
Member

parkera commented Feb 2, 2017

Let's certainly cherry-pick your fix in #843 to master though.

@UberJason
Copy link
Contributor

Hello, did the cherry-pick of #843 into master end up happening? The latest snapshot (02-05) still seems to fail to build CoreFoundation - at least, a Vapor project I'm working with fails to import Foundation with the same errors as those referenced in #843.

@CodaFi
Copy link
Member

CodaFi commented Feb 6, 2017

Cherry Pick is happening in #868.

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

7 participants