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

faraday 2 support #155

Merged
merged 3 commits into from
Feb 28, 2022
Merged

faraday 2 support #155

merged 3 commits into from
Feb 28, 2022

Conversation

irphilli
Copy link
Owner

@irphilli irphilli commented Feb 25, 2022

This brings in faraday 2.0 support by:

  • Dropping faraday_middleware. Common middleware now bundled with newer versions of Faraday.
  • Simplifying our dependencies by dropping excon. Users can still choose to use that adapter with options, but we will default to net_http.
  • Requiring Faraday >= 1.10. Faraday 1.10 is the lowest version on the 1.x branch that has most of the middleware we need bundled in.
  • Updating our build matrix to test against multiple versions of Faraday.

This similifies our dependencies a bit.
Users can still opt to use the excon adapter if they wish.
@irphilli irphilli force-pushed the chore/faraday-2-support branch 2 times, most recently from 283e550 to b283df8 Compare February 25, 2022 17:47
- Remove faraday-middleware as most middleware now bundled with faraday
- Require Faraday 1.10+ (1.10 adds json middleware used by us)
@irphilli irphilli force-pushed the chore/faraday-2-support branch from b283df8 to 7dc744f Compare February 25, 2022 17:53
@codeclimate
Copy link

codeclimate bot commented Feb 25, 2022

Code Climate has analyzed commit 7dc744f and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 97.5% (0.0% change).

View more on Code Climate.

@irphilli irphilli merged commit d566e88 into master Feb 28, 2022
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.

1 participant