Skip to content
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

Add getChanges method #42

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add getChanges method #42

wants to merge 8 commits into from

Conversation

derUli
Copy link

@derUli derUli commented Mar 15, 2019

Unfortunately I wasn't able to get your tests running on my windoze machine.

But I verified my code with a standalone php script.

C:\Users\deruli\Documents\GitHub\git-php>php test.php
array(1) {
  ["src/GitRepository.php"]=>
  string(1) "M"
}

@janpecha
Copy link
Contributor

Thanks! Looks good. But I'm not sure about API, give me some time, I will think about it.

BTW. What is your use-case?

@derUli
Copy link
Author

derUli commented Mar 21, 2019

@janpecha
I am the main developer of a content management system.
I developed a git client extension module for it using your lib
https://github.com/derUli/git_client

@janpecha
Copy link
Contributor

Ok, thanks.

@janpecha janpecha added this to the Version 4.0 milestone Jan 15, 2020
@derUli
Copy link
Author

derUli commented Jan 22, 2020

Why is this merge request still not merged?

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