-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Pipe it to connections #73688
Pipe it to connections #73688
Conversation
added 'defaults' entry for ini pipelining from ssh plugin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look good. Can we add unit tests for this? The local connection plugin doesn't have very good coverage currently, though it does at least have a very basic unit test.
@sdoran the changes are coverd in generic set_options/get_option tests, i would not add a unit test per option per plugin. |
* pipelining tweaks added 'defaults' entry for ini pipelining from ssh plugin (cherry picked from commit 9690512)
moving pipeline config to the specific plugins.
also fixes #73672
ISSUE TYPE
COMPONENT NAME
connection