Davide Spadini

Actually, I just noticed that in the documentation you say "context diff", not "combined". At this point I think this is more of a feature request than an issue :)

Awesome 😄 what I'd need for PyDriller is: `old-path`, `new-path`, `modification type (added, deleted, etc.)`, and `diff`. So it shouldn't be too difficult to get this. We can start from...

Oh no worries! I know the feeling 😄

Thanks for raising the issue! I just checked and this code works: ``` from pydriller import Repository for commit in Repository("test-repos/small_repo", only_no_merge=True).traverse_commits(): print(commit.lines) print(commit.files) ``` but this code doesn't work:...

The reason is that the object is inside a context manager, so it will get destroyed after the loop. I don't see a good solution to this, other than just...

Oh that's true! I'll try and make it available there too!

Hi! In the commit you showed, the file in question is renamed, hence there are both the old path and the new path. A small test: ``` for commit in...

Oh thanks, now I saw it. Indeed, in this specific case, looks like GitPython returns the wrong information: ``` %s ========== lhs: None rhs: None file renamed from 'tests/integration/__init__.py' file...

I was able to reproduce locally: ``` cd /tmp mkdir test && cd test git init touch asd.txt git add asd.txt git commit -m "add empty file" ``` Then run:...

Indeed I remembered this kind of problem was raised before. Turns out that 4 years ago I opened a PR about this: https://github.com/gitpython-developers/GitPython/issues/749 I never managed to work on it.