8000 Fixes CI by ashfurrow · Pull Request #667 · Moya/Moya · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fixes CI #667

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 5 commits into from
Sep 22, 2016
Merged

Fixes CI #667

merged 5 commits into from
Sep 22, 2016

Conversation

ashfurrow
Copy link
Member

Fixes #665. Also excludes the Demo app from our code coverage report.

@ashfurrow
Copy link
Member Author

Trying to split up Carthage validation onto Travis, so they can run in parallel and save build time. We'll see!

fail("next called for invalid data")
case .failed(let error):
case .failure(let error):
receivedError = error
Copy link
Member Author
@ashfurrow ashfurrow Sep 22, 2016

Choose a reason for hiding this comment

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

Do these changes make sense to you, @justinmakaila? I don't want to do something non-idiomatic to ReactiveSwift.

@ashfurrow
Copy link
Member Author

Nah, never mind. Travis is having outages, might be worth reconsidering separately, but not now.

@ashfurrow
Copy link
Member Author

Danger failing again, gotta learn not to force-push to PRs...

@ashfurrow ashfurrow merged commit 1df10be into master Sep 22, 2016
@ashfurrow ashfurrow deleted the ci-fix branch September 22, 2016 01:36
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.

1 participant
0