8000 examples: add config example back. Fixes #523 by cmiller01 · Pull Request #557 · go-git/go-git · GitHub
[go: up one dir, main page]

Skip to content

examples: add config example back. Fixes #523 #557

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

cmiller01
Copy link
Contributor

* previously an example existed for working with `config.Config`, but changes in implementation made the example not usable. (see go-git#75)
* this mostly puts back the original example with some modifications to
make it work with current implementation
@cmiller01 cmiller01 changed the title examples: add config example. Fixes #523 examples: add config example back Jul 26, 2022
@cmiller01 cmiller01 changed the title examples: add config example back examples: add config example back. Fixes #523 Jul 26, 2022
@mcolburn
Copy link

Please add a config example. It lists in the README for the examples, but does not exist.

@cmiller01
Copy link
Contributor Author

Please add a config example. It lists in the README for the examples, but does not exist.

Coming back to some of my open PRs! This is the config example, it's in the README now (like you mention: https://github.com/go-git/go-git/tree/master/_examples) but the config example is missing, this should put it back

@pjbgf
Copy link
Member
pjbgf commented Aug 6, 2024

@cmiller01 do you mind rebasing your PR and reintroducing the link on _examples/README.md?

9C9A

@pjbgf
Copy link
Member
pjbgf commented Dec 18, 2024

Closing this - I will forward port the commit directly to v6-exp instead.

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
0