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

chore: bump faraday + faraday_middleware version range #56

Merged
merged 2 commits into from Jan 20, 2020

Conversation

samspills
Copy link
Contributor

Faraday released version 1.0.0 on Jan 1 2020. I updated the faraday and faraday_middleware dependencies, and bumped the mrkt version from 0.11.0 to 0.11.1. I ran tests with bundle exec rspec and everything looks good.

Closes #52

faraday has released major version 1.0.0 (Jan 1 2020)
faraday_middleware range was matching faraday, but set to be inline with its own
releases (0.13.1, Feb 4 2019)
@raszi
Copy link
Owner

raszi commented Jan 9, 2020

I was planning to wait until faraday_middleware starts supporting 1.0.

@samspills
Copy link
Contributor Author

Ahh, I see. This is kind of a blocker for my team right now. Could I change the Faraday dependency to <= 0.17.3 for now? Long story short: we use the mrkt gem, but we also need to be on faraday 0.16.2. I figured I would just bump to the most recent faraday version but you make a good point about the middleware gem!

faraday_middleware doesn't support Faraday v1.0.0 yet
@samspills
Copy link
Contributor Author

Hey @raszi, any thoughts about bumping to 0.17.3? As I mentioned this is a weird blocker for me right now. If you have any particular concerns I can add more tests to further verify that the new faraday version functions as expected?

@raszi raszi merged commit b00e7ff into raszi:master Jan 20, 2020
@raszi raszi changed the title Bump faraday + faraday_middleware version range chore: bump faraday + faraday_middleware version range Jan 20, 2020
@samspills samspills deleted the samspills/bump-faraday-version branch January 20, 2020 14:31
@raszi
Copy link
Owner

raszi commented Jan 20, 2020

Released as 0.11.1.

@samspills
Copy link
Contributor Author

Thank you!!

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.

Update Faraday Dependency
2 participants