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

Added support for Faraday 2 #152

Merged
merged 2 commits into from
Mar 3, 2023
Merged

Conversation

igor-alexandrov
Copy link
Contributor

This PR adds support for Faraday 2 to asana gem (#141).

All specs pass, I've also tested this in my project and everything works well.

Let's discuss.

@jv-asana
Copy link
Contributor

jv-asana commented Mar 2, 2023

@igor-alexandrov Thank you for sending this PR. Do you also mind updating the Appraisals file. I think the GitHub action builds are failing because of it.

@igor-alexandrov
Copy link
Contributor Author

@jv-asana added two latest faraday version to Appraisals.

Copy link
Contributor

@jv-asana jv-asana left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you for submitting this PR @igor-alexandrov.

@jv-asana jv-asana merged commit 962f175 into Asana:master Mar 3, 2023
@igor-alexandrov
Copy link
Contributor Author

@jv-asana will you also release gem to Rubygems?

@igor-alexandrov igor-alexandrov deleted the feature/faraday-2 branch March 4, 2023 11:31
@jv-asana
Copy link
Contributor

jv-asana commented Mar 6, 2023

@jv-asana will you also release gem to Rubygems?

Hi @igor-alexandrov I was going to wait until we update our library with new endpoints but I can go ahead and release it now instead of waiting.

@igor-alexandrov
Copy link
Contributor Author

@jv-asana no reason to rush, release it when you consider the right time to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants