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

Update Faraday dependency to < 0.14 #318

Merged
merged 1 commit into from Dec 15, 2017

Conversation

zacharywelch
Copy link
Contributor

Fixes #315
Latest version of faraday is 0.13.1

@coveralls
Copy link

coveralls commented Sep 26, 2017

Coverage Status

Coverage remained the same at 96.0% when pulling ba65f82 on zacharywelch:zw/update-faraday-0.13.1 into 452a085 on intridea:master.

@sienatime
Copy link

+1 for this. Looks like failures are related to Rubocop, which seems odd since you didn't change any code. Is that a known issue?

@ismasan
Copy link

ismasan commented Dec 13, 2017

+1 for this. Anything I can do to help get it done?

@pboling
Copy link
Member

pboling commented Dec 13, 2017

@zacharywelch can you run bundle exec rubocop -a to fix the rubocop violations? I don't want to break the build on master.

@ismasan you can redo this PR change (and resubmit) with Rubocop fixes so the build goes green!

@itkrt2y itkrt2y mentioned this pull request Dec 14, 2017
@zacharywelch zacharywelch force-pushed the zw/update-faraday-0.13.1 branch 3 times, most recently from 1011d56 to ba65f82 Compare December 14, 2017 03:59
@zacharywelch
Copy link
Contributor Author

Hi @pboling, the build still fails even after autocorrect because of rubocop differences. Are you ok w/ the rubocop fixes in #322?

@pboling
Copy link
Member

pboling commented Dec 15, 2017

@zacharywelch thanks for the heads up on that PR. Merged it. We are now good here. I'm a new owner/maintainer, just getting familiar with the state of the project, trying to revive it.

(All help is appreciated!)

@pboling pboling merged commit 9fd846a into oauth-xx:master Dec 15, 2017
@zacharywelch zacharywelch deleted the zw/update-faraday-0.13.1 branch December 15, 2017 14:40
@josephpage josephpage added this to the 1.5.0 milestone Jan 9, 2018
@pboling pboling added the in Changelog Has been added to Changelog label Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in Changelog Has been added to Changelog
Projects
None yet
7 participants