8000 examples: config, Show how to save config changes by lcostea · Pull Request #31 · go-git/go-git · GitHub
[go: up one dir, main page]

Skip to content

examples: config, Show how to save config changes #31

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

Merged
merged 1 commit into from
Apr 18, 2020
Merged

examples: config, Show how to save config changes #31

merged 1 commit into from
Apr 18, 2020

Conversation

lcostea
Copy link
Contributor
@lcostea lcostea commented Apr 18, 2020

It took me a few hours and going through the code of go-git to figure out I need to call SetConfig in order for the config changes to be applied to the local config file.
Especially that when doing CreateRemote the call to SetConfig is actually done behind the scene so the remote appeared in the file. While for the user local settings I had to actually call SetConfig myself.
Hope this will save someone a few hours or at least minutes of troubleshooting.

< 8000 input type="hidden" name="dropdown_direction" value="w" autocomplete="off" data-targets="batch-deferred-content.inputs" />
@mcuadros mcuadros merged commit 53f8784 into go-git:master Apr 18, 2020
@lcostea lcostea deleted the liviu/add_save_config_changes branch April 18, 2020 16:47
mcuadros added a commit to mcuadros/go-git that referenced this pull request May 24, 2020
…ig_changes"

This reverts commit 53f8784, reversing
changes made to bb3a1bf.
traidare pushed a commit to traidare/go-git that referenced this pull request Oct 26, 2024
examples: config, Show how to save config changes
traidare pushed a commit to traidare/go-git that referenced this pull request Oct 26, 2024
…ig_changes"

This reverts commit 5ffb884, reversing
changes made to a07e17a.
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.

2 participants
0