Skip to content

change FileManager.default() class function to static let #484

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

Merged
merged 3 commits into from
Aug 2, 2016

Conversation

jpsim
Copy link
Contributor

@jpsim jpsim commented Jul 27, 2016

to match Darwin version as of Xcode 8

@parkera
Copy link
Contributor

parkera commented Jul 27, 2016

Looks good.

@parkera
Copy link
Contributor

parkera commented Jul 27, 2016

@swift-ci please test

@CodaFi
Copy link
Contributor

CodaFi commented Jul 31, 2016

@jpsim This needs rebasing.

@jpsim jpsim force-pushed the filemanager-default branch from af1f082 to c5e4e6b Compare July 31, 2016 23:11
@jpsim
Copy link
Contributor Author

jpsim commented Jul 31, 2016

rebased

@CodaFi
Copy link
Contributor

CodaFi commented Jul 31, 2016

@swift-ci please test.

@CodaFi
Copy link
Contributor

CodaFi commented Aug 1, 2016

@jpsim Looks like you missed something in NSTask.swift.

@jpsim
Copy link
Contributor Author

jpsim commented Aug 1, 2016

Indeed. Fixed.

@CodaFi
Copy link
Contributor

CodaFi commented Aug 1, 2016

Once more, with feeling.

@swift-ci please test.

@jpsim
Copy link
Contributor Author

jpsim commented Aug 1, 2016

This should be merged with swiftlang/swift-package-manager#584

@ddunbar
Copy link
Contributor

ddunbar commented Aug 2, 2016

Unfortunately this has been conflicted... @jpsim I can shepherd landing this (swift-ci can't handle multi-repo PRs yet) if someone rebases it.

@jpsim jpsim force-pushed the filemanager-default branch from ee5bb25 to df452e8 Compare August 2, 2016 17:27
@jpsim
Copy link
Contributor Author

jpsim commented Aug 2, 2016

@ddunbar rebased again 😅

@ddunbar
Copy link
Contributor

ddunbar commented Aug 2, 2016

Thanks, testing now...

@ddunbar ddunbar merged commit df452e8 into swiftlang:master Aug 2, 2016
@ddunbar
Copy link
Contributor

ddunbar commented Aug 2, 2016

Merged, thank you!

@jpsim jpsim deleted the filemanager-default branch August 2, 2016 17:44
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.

4 participants