wert007 / commit-analyzer

Gets the time somebody worked on something from `git log`
MIT License
1 stars 1 forks source link

[Enhancement] [Bugfix] Grab Git History Automatically #17

Closed kevinmatthes closed 2 years ago

kevinmatthes commented 2 years ago

This patch introduces the automatic reading from Git itself. It thereby fixes #16 and replaces the feature request from #3.

wert007 commented 2 years ago

I would also squash the last 4 commits and the first two! While you have now (almost) all changes in one commit each, I think, those changes could still be considered too small sub parts of the actual problem. At least in my opinion. 😅

kevinmatthes commented 2 years ago

Since this is a branch on its own, merging on squash will combine all of them to a single one.

wert007 commented 2 years ago

While this is true, I would prefer if you tried to improve your commit sizes. It makes reviewing easier, especially, if you do it on per commit basis, and it would also be nice to simply merge the commits in to the main branch, to have a better overview over the commits. Ironically, tools like this, would profit from this. 😄

kevinmatthes commented 2 years ago

Users now need to specify a source to read from. This might be either the local Git history (--git) or an input file (-i). stdin will be added later, perhaps with an option, as well, such that a call without options will redirect to the help overview.