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

Introduce GDB::Git::Wrapper #3

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@sergeyromanov
Contributor

sergeyromanov commented Oct 25, 2016

I've just mimicked Git::Repository backend. The ObjectReader's methods are not there yet, but this should be a start.

@sergeyromanov

This comment has been minimized.

Show comment
Hide comment
@sergeyromanov

sergeyromanov Oct 25, 2016

Contributor

777cfe6 is really just a little discrepancy fix, I can re-apply in a different branch if you'd like so :)

Contributor

sergeyromanov commented Oct 25, 2016

777cfe6 is really just a little discrepancy fix, I can re-apply in a different branch if you'd like so :)

@book

This comment has been minimized.

Show comment
Hide comment
@book

book Nov 3, 2016

Owner

I actually had a git-wrapper branch already with the ObjectReader role, but hadn't pushed it yet. I've pushed it now, and I'll integrate your additions soon. They seem to complement nicely.

Owner

book commented Nov 3, 2016

I actually had a git-wrapper branch already with the ObjectReader role, but hadn't pushed it yet. I've pushed it now, and I'll integrate your additions soon. They seem to complement nicely.

@book

This comment has been minimized.

Show comment
Hide comment
@book

book Nov 3, 2016

Owner

I've applied 777cfe6 independently as 8f27d21. Thanks!

Owner

book commented Nov 3, 2016

I've applied 777cfe6 independently as 8f27d21. Thanks!

@book

This comment has been minimized.

Show comment
Hide comment
@book

book Nov 10, 2016

Owner

I've applied 397aff4 as 78ab99a.

Owner

book commented Nov 10, 2016

I've applied 397aff4 as 78ab99a.

@book

This comment has been minimized.

Show comment
Hide comment
@book

book Nov 10, 2016

Owner

With both those commits applied to the main repository, I'm now closing this PR. Thanks to your patches, Git::Database::Backend::Git::Wrapper will be part of the next release.

Owner

book commented Nov 10, 2016

With both those commits applied to the main repository, I'm now closing this PR. Thanks to your patches, Git::Database::Backend::Git::Wrapper will be part of the next release.

@book book closed this Nov 10, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment