-
Notifications
You must be signed in to change notification settings - Fork 252
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
Support Co-authored Commits #70
Comments
It seems the current github stats doesn't include the co-authored commits and is same as what's printed by Repo : https://github.com/MichaelSafwatHanna/FitnessApp
@YoussefRaafatNasry |
I am seeing the same thing @ProcrastinatorCp is seeing with regards to how GitHub is parsing the stats given the current information. That said, I think this feature in general is a good idea. It might be worth looking into supporting the Linux kernel commit message conventions by also supporting the following:
https://git.wiki.kernel.org/index.php/CommitMessageConventions |
Agree. Good to have this feature. |
git-quick-stats#70 Commits by multiple authors were not counted in co-author's contribution. Modified the commitsPerAuthor() function to include the commits which have "Co-authored-by:" in the commit message.
git-quick-stats#70 Commits by multiple authors were only counted towards the main authors commits. Modified the commitsPerAuthor function to include co-authors as well.
git-quick-stats#70 Commits by multiple authors (co-authored-by) were only counted towards main author. Modified commitsPerAuthor() to include co-author's commits.
9) Git commits per author
gives the following output:While the GitHub repo contains the following stats due to Co-authors.
git shortlog --grep='Co-authored-by: Youssef Raafat' -s -n
the output will be77 Michael Safwat
.shortlog
to the 204 commits of the previous output gives us something near GitHub stats.So is it possible to add option to count commits by authors & co-authors?
The text was updated successfully, but these errors were encountered: