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

eliminate a faraday 1.x deprecation warning #385

Merged

Conversation

skaes
Copy link
Contributor

@skaes skaes commented Jun 7, 2022

Copy link
Member

@tnir tnir left a comment

Choose a reason for hiding this comment

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

It looks like changing here work well to me.

@skaes
Copy link
Contributor Author

skaes commented Jun 8, 2022

@tnir what change to the code do you suggest? I can't find any.

@tnir
Copy link
Member

tnir commented Jun 8, 2022

I thought I would do that after I become a maintainer and release the gem.

@skaes skaes force-pushed the eliminate-faraday-basic-auth-deprecation-warning branch from 52c0adc to 5c2ac96 Compare June 19, 2022 10:21
@tnir tnir added this to the v2.0.0 milestone Jun 20, 2022
@tnir tnir modified the milestones: v2.0.0, v2.1.0 Jun 27, 2022
@tnir
Copy link
Member

tnir commented Jun 27, 2022

Moving this forward upcoming milestone.

@skaes skaes marked this pull request as draft June 28, 2022 04:28
@skaes
Copy link
Contributor Author

skaes commented Jun 28, 2022

I'll try to fix the Ruby 2.2.10 problem.

@skaes skaes force-pushed the eliminate-faraday-basic-auth-deprecation-warning branch 4 times, most recently from 982a0c0 to 05dcbc1 Compare June 28, 2022 05:02
@skaes skaes force-pushed the eliminate-faraday-basic-auth-deprecation-warning branch from 05dcbc1 to 3ec302e Compare June 28, 2022 05:05
@skaes skaes marked this pull request as ready for review June 28, 2022 05:06
@skaes skaes requested a review from tnir June 28, 2022 05:07
@tnir tnir self-assigned this Jun 28, 2022
@skaes
Copy link
Contributor Author

skaes commented Jun 28, 2022

@tnir sorry for the confusion. Should be complete by now.

Copy link
Member

@tnir tnir left a comment

Choose a reason for hiding this comment

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

Now it looks like everything works well 🎉 Thanks @skaes 🙏

@tnir tnir merged commit 71a69c2 into geminabox:master Jun 28, 2022
@skaes skaes deleted the eliminate-faraday-basic-auth-deprecation-warning branch June 28, 2022 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants