Skip to content

Relax faraday and faraday_middleware version #364

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 1 commit into from
Aug 17, 2020

Conversation

sikachu
Copy link
Contributor

@sikachu sikachu commented Apr 7, 2020

Hello!

Faraday and faraday_middleware has recently gone 1.0:

We want to upgrade a few gems on our project to the latest version, but found out that there's a dependency conflict because json_api_client has strict faraday and faraday_middleware requirement.

The API hasn't changed much since 0.9 -> 1.0, and the test still pass, so I think we should be good to go.

Please let me know if you have any questions or feedback about this PR. Thank you!

@rahearn
Copy link

rahearn commented Jun 8, 2020

Can this be merged? The TravisCI tests passed, not sure why the status check hasn't updated.

@omarsotillo
Copy link

Hi @lcpriest @gaorlov! 👋 We found the same situation on our environments.

Could you help on the review here? 👯

Thanks for the cool gem 💎

@lcpriest
Copy link
Contributor

I'm not 100% on this, we might need a minor version update.

Thoughts @gaorlov?

@rahearn
Copy link

rahearn commented Jun 19, 2020

FWIW, I'm using this branch in production and it seems to be working fine.

@gaorlov
Copy link
Collaborator

gaorlov commented Jun 20, 2020

Thanks do much for the contribution! It looks like 0.17 -> 1.0 is additive. Can you confirm that there are no breaking changes and I'll merge it in.

Thanks!

Greg

@sikachu
Copy link
Contributor Author

sikachu commented Aug 3, 2020

@gaorlov sorry for the delay.

Yes, 0.17 -> 1.0 is just an additive change. We are using this in production and we found no breaking changes.

Please feel free to merge in at your convenience. Thanks!

@iMacTia
Copy link

iMacTia commented Aug 17, 2020

@gaorlov I can confirm what was already said, when we worked on Faraday 1.0 the aim was to make it backwards-compatible with versions 0.17.x, and we postponed all the breaking changes to v2.0
That said, we took some freedom dropping a few old Rubies and updated some gems, but if your tests suite is all green then it shouldn't be an issue.

I've just found this gem and I think it makes a great use of Faraday, so I'd love to see this PR merged

@gaorlov gaorlov merged commit 2c7e75b into JsonApiClient:master Aug 17, 2020
@gaorlov
Copy link
Collaborator

gaorlov commented Aug 17, 2020

@sikachu @rahearn @omarsotillo @lcpriest @iMacTia 1.17.0 is now in live. Thanks for your contributions and patience!

@iMacTia
Copy link

iMacTia commented Aug 17, 2020

Legend 🎉 !

@sikachu sikachu deleted the relax-faraday-versions branch August 25, 2020 08:26
@sikachu sikachu restored the relax-faraday-versions branch August 27, 2020 12:49
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.

6 participants