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

Fixes #2185 Follow up on dependency lib version needed #2186

Merged
merged 4 commits into from
Jan 17, 2024

Conversation

pawelkeska
Copy link
Contributor

@sidz ClientInterface was added from 8.4.1 i've changed composer.json

@Rufin i added some tests to connection and fix bug with path in Client

@pawelkeska pawelkeska changed the title [#2185] Follow up on dependency lib version needed #2185 Follow up on dependency lib version needed Jan 17, 2024
@pawelkeska pawelkeska changed the title #2185 Follow up on dependency lib version needed Fixes #2185 Follow up on dependency lib version needed Jan 17, 2024
Copy link
Contributor

@sidz sidz left a comment

Choose a reason for hiding this comment

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

Brilliant. thanks @pawelkeska 👍

@@ -167,14 +167,6 @@ public function getTransportObject(): Transport
return $this->getTransport();
}

/**
Copy link
Owner

Choose a reason for hiding this comment

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

This goes away because it is not supported anymore?

@ruflin ruflin mentioned this pull request Jan 17, 2024
6 tasks
Copy link
Owner

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Change LGTM. Before we merge, quickly get some details on why some parts are removed (I guess not supported anymore)

@pawelkeska
Copy link
Contributor Author

@ruflin yes, isPersistent and getAuthType was used by transports elastica. It's not anymore need.

@ruflin ruflin merged commit 32a3010 into ruflin:8.x Jan 17, 2024
15 checks passed
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