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

bumped ruby version to 3 and faraday version to 2 #44

Merged

Conversation

obroshnij
Copy link

This PR adds support for ruby v.3.2 and faraday version 2 (lostisland/faraday#1358) which is shipped with a bunch of optimizations and improvements out of the box

bumped faraday to latest version and included adapters for testing

remove byebug and small fixes

added ruby 3.2 to circleci file

install libcurl during CI
em_synchrony: Faraday::Adapter::EMSynchrony,
em_http: Faraday::Adapter::EMHttp,
# em_synchrony: Faraday::Adapter::EMSynchrony,
# em_http: Faraday::Adapter::EMHttp,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we just rm?

# No updates since 2021, the method `dependecy` from the class Faraday::Adapter has been removed
# https://github.com/lostisland/faraday/blob/main/UPGRADING.md#the-dependency-method-in-middlewares-has-been-removed
# require 'faraday/em_http'
# require 'faraday/em_synchrony'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we just rm?

test/lib/adapter_test.rb Outdated Show resolved Hide resolved
test/lib/adapter_test.rb Outdated Show resolved Hide resolved
@UmanShahzad UmanShahzad merged commit 4e31297 into ipinfo:master Nov 21, 2023
0 of 5 checks passed
@obroshnij
Copy link
Author

thanks @UmanShahzad

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.

3 participants