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

Platform: Fix faraday version #9

Closed
wants to merge 2 commits into from
Closed

Conversation

adamof
Copy link
Contributor

@adamof adamof commented Jul 17, 2014

Faraday introduced a breaking change in 0.8.9 that changes the way the faraday request options are set.

@adamof
Copy link
Contributor Author

adamof commented Jul 17, 2014

@mezis do you mind taking a look, and pushing a new version of the gem if this seems fine?

cc @krisleech

@krisleech
Copy link
Contributor

I can confirm this work happily in an app which has an existing Faraday dependency as such:

gem 'faraday', '>= 0.8.0'

@tadejm
Copy link

tadejm commented Jul 18, 2014

Shouldn't we ride latest version and update how requests are made?

@andycroll
Copy link

I'm for 'staying up to date', particularly for SOA apps.

@adamof
Copy link
Contributor Author

adamof commented Jul 18, 2014

I agree, will take care later / over the weekend

@adamof adamof closed this Jul 18, 2014
@mezis mezis mentioned this pull request Jul 23, 2014
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