Skip to content

Add getChanges method#42

Open
derUli wants to merge 8 commits into
czproject:masterfrom
derUli:master
Open

Add getChanges method#42
derUli wants to merge 8 commits into
czproject:masterfrom
derUli:master

Conversation

@derUli

@derUli derUli commented Mar 15, 2019

Copy link
Copy Markdown

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
Copy Markdown
Member

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

derUli commented Mar 21, 2019

Copy link
Copy Markdown
Author

@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
Copy Markdown
Member

Ok, thanks.

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

derUli commented Jan 22, 2020

Copy link
Copy Markdown
Author

Why is this merge request still not merged?

@wacki4

wacki4 commented Nov 28, 2025

Copy link
Copy Markdown

Also, waiting for this option.

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