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

Add compatibility for recent versions of Faraday #39

Merged
merged 1 commit into from
Feb 26, 2021
Merged

Add compatibility for recent versions of Faraday #39

merged 1 commit into from
Feb 26, 2021

Conversation

FraDim
Copy link
Contributor

@FraDim FraDim commented Feb 22, 2021

Allow the use of a Faraday version that supports ruby 2.3, while also allowing more recent versions of Faraday.
This will increase repo compatibility and enable the users of this library to follow the suggestion of using its latest version.

Allow the use of a Faraday version that supports ruby 2.3, while also allowing more recent versions of Faraday.
This will increase repo compatibility and enable the users of this library to follow the suggestion of using its latest version.
@FraDim
Copy link
Contributor Author

FraDim commented Feb 22, 2021

My app uses Faraday v1.3.0 (the latest at the time of writing) but cannot use the latest version of the Aftership gem, because of the very tight Faraday version constraint. I think this will improve the situation that caused the revert of #35.

@Bossa573 @renyuanz I haven't seen a way to run automated tests, so can you please check if this works for your supported Ruby versions?

@Bossa573
Copy link
Member

@FraDim thank you for the PR, and we will merge this PR and release a new version

@Bossa573 Bossa573 merged commit 9728f6a into AfterShip:master Feb 26, 2021
@FraDim FraDim deleted the patch-1 branch February 26, 2021 11:52
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.

None yet

2 participants