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 type of change to files_dict of a commit #1943

Conversation

JonasScharpf
Copy link
Contributor

This allows to not only get the total, inserted or deleted number of lines being changed but also the type of change like Added (A), Copied (C), Deleted (D), Modified (M), Renamed (R), type changed (T), Unmerged (U), Unknown (X), or pairing Broken (B)

This allows to not only get the total, inserted or deleted number of
lines being changed but also the type of change like
Added (A), Copied (C), Deleted (D), Modified (M), Renamed (R),
type changed (T), Unmerged (U), Unknown (X), or pairing Broken (B)
@JonasScharpf JonasScharpf force-pushed the feature/add-change-type-to-commit-stats branch from 323c042 to ce8a69a Compare July 17, 2024 12:10
Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for contributing this improvement. It really does look great already.

There is one question left, could you take a look?
Thanks

test/test_stats.py Show resolved Hide resolved
Copy link
Member

@Byron Byron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for contributing, much appreciated.

test/test_stats.py Show resolved Hide resolved
@Byron Byron merged commit ea583e1 into gitpython-developers:main Jul 18, 2024
26 checks passed
@JonasScharpf JonasScharpf deleted the feature/add-change-type-to-commit-stats branch July 18, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants