Skip to content

Support multiple --config options in Git.clone #470

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

Closed
wants to merge 1 commit into from

Conversation

svl7
Copy link

@svl7 svl7 commented Jun 5, 2020

Makes opts[:config] in Git.clone accept an Array of = pairs
so that multiple --config options can be passed to the clone command.
A single config entry can still be passed as a String.

Signed-off-by: Marco Rüdisüli git@rudisu.li

Makes opts[:config] in Git.clone accept an Array of <key>=<value> pairs
so that multiple --config options can be passed to the clone command.
A single config entry can still be passed as a String.

Signed-off-by: Marco Rüdisüli <git@rudisu.li>
Signed-off-by: Marco Rüdisüli <Marco.Ruedisueli@swisscom.com>
@stale
Copy link

stale bot commented Aug 8, 2020

A friendly reminder that this issue had no activity for 60 days.

@stale stale bot added the stale label Aug 8, 2020
@svl7
Copy link
Author

svl7 commented Aug 10, 2020

Just here because I got the stale bot message, maybe a maintainer can merge or decline this? @jcouball

@stale stale bot removed the stale label Aug 10, 2020
@stale
Copy link

stale bot commented Oct 11, 2020

A friendly reminder that this issue had no activity for 60 days.

@stale stale bot added the stale label Oct 11, 2020
@bond-os
Copy link

bond-os commented Oct 13, 2020

It would be nice with this PR being merged to next release.

@stale stale bot removed the stale label Oct 13, 2020
@stale
Copy link

stale bot commented Dec 25, 2020

A friendly reminder that this issue had no activity for 60 days.

@stale stale bot added the stale label Dec 25, 2020
@jcouball
Copy link
Member

jcouball commented Mar 7, 2023

Since I need to make a change or two to the implementation and documentation, I have re-implemented this PR in #653

@jcouball jcouball closed this Mar 7, 2023
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