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

v0.16.0 contains breaking changes from 1.0.RC-1 #1031

Closed
rlafferty opened this issue Sep 27, 2019 · 4 comments
Closed

v0.16.0 contains breaking changes from 1.0.RC-1 #1031

rlafferty opened this issue Sep 27, 2019 · 4 comments

Comments

@rlafferty
Copy link

rlafferty commented Sep 27, 2019

Basic Info

  • Faraday Version: 0.16.0
  • Ruby Version: 2.6.X

Issue description

RC-1 changes the namespacing on errors. These changes have been included in the release of 0.16.0, this is a backwards incompatible change, is it expected in this version or should it be held until 1.0.0?

Steps to reproduce

[1] pry(main)> require 'faraday'
=> true
[2] pry(main)> Faraday::Error::RetriableResponse.new('oh noes')
NameError: uninitialized constant Faraday::Error::RetriableResponse
Did you mean?  Faraday::RetriableResponse
from (pry):2:in `__pry__'
[3] pry(main)> Faraday::RetriableResponse.new('oh noes')
=> #<Faraday::RetriableResponse #<Faraday::RetriableResponse: oh noes>>
@rlafferty
Copy link
Author

Seems #1030 may address this

@rlafferty rlafferty changed the title v0.16.0 contains breaking changes from RC-1 v0.16.0 contains breaking changes from 1.0.RC-1 Sep 27, 2019
@stuyam
Copy link

stuyam commented Sep 27, 2019

Yeah it seems to be a breaking change for a dependency that Octokit uses and is breaking Octokit: octokit/octokit.rb#1155

@technoweenie
Copy link
Member

Thanks for the report. If #1030 is good, we should be able to roll out v0.16.1 soon.

@rlafferty
Copy link
Author

Thanks for the report. If #1030 is good, we should be able to roll out v0.16.1 soon.

Thanks for the super quick turn around @technoweenie !! 🎉

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

No branches or pull requests

3 participants