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

Implemented adapter_client_setup #203

Closed
wants to merge 1 commit into from
Closed

Conversation

Wardrop
Copy link

@Wardrop Wardrop commented Apr 29, 2019

As per issue #202

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.09%) to 96.976% when pulling a59d9a0 on Wardrop:master into 2fbd6b1 on savonrb:master.

@ioquatix
Copy link
Contributor

ioquatix commented Oct 6, 2021

I think this is acceptable but can you please rebase on master?

@pcai
Copy link
Member

pcai commented Oct 1, 2022

Hey @Wardrop - thanks for the contribution! We haven't heard from you, so I tried to rebase this commit in another branch, and it failed the specs due to the global state persisting between tests. Fixing now myself to get it into main branch and will circle back once it is merged.

@pcai
Copy link
Member

pcai commented Oct 1, 2022

merged in as #237

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

Successfully merging this pull request may close these issues.

5 participants