search

ejwa / gitinspector

:bar_chart: The statistical analysis tool for git repositories
GNU General Public License v3.0
2.38k stars 328 forks source link

commit count incosistent with github one #182

Closed typekpb closed 6 years ago

typekpb commented 6 years ago

I've noticed, commit count in gitinspector doesn't reflect the reality => too low.

Steps to reproduce: Run gitspector on this project: github stats: image

gitinspector stats: image

it seems:

Where is the difference gone?

typekpb commented 6 years ago

btw. the following opts have been used during the test run:-mrTw

adam-waldenberg commented 6 years ago

Hi @typekpb. This is the correct behavior. As stated by the documentation, gitinspector only counts commits that include any of the filetypes currently being scanned. Everything else is filtered out. This includes all statistics gitinspector shows you, not only the commit count.

Take, @bartvanandel as an example. He has 6 commits in the repo (not 5 as claimed by GitHub), but only one of them actually consists of any code; https://github.com/ejwa/gitinspector/commit/3d11cdce44711d5f0af56d058689706a2e4ed579

Also notice that gitinspector only counts the 29 insertions from that filetype. Because from all the files @bartvanandel has commited it is only tracking the javascript ones.

So in gitinspectors eyes (with the default filetypes), he only has one valid commit and 29 valid lines/insertions.

If you want to track text files and everything else - please refer to the documentation and FAQ. It should help you with most questions.

Closing this issue.

typekpb commented 6 years ago

@adam-waldenberg thank you for clarificatoin, -f "**" did the trick for me