gitinspector icon indicating copy to clipboard operation
gitinspector copied to clipboard

Add new "-M" option, so that -wM would produce both weekly and monthly

Open sobomax opened this issue 9 years ago • 3 comments

Add new "-M" option, which when used in conjunction with "-w" will cause gitinspector output both monthly and weekly stats for the period in question. It is useful in some cases to have both.

sobomax avatar Feb 09 '16 09:02 sobomax

Hi @sobomax.

Yes, there might be some credence to adding this. I'm not completely convinced, but it is definitely worth considering.

Some problems with the pull request, before this can even be considered for merging;

  • Needs "git config" support (see config.py), which also means we need a long option for this.
  • How do we handle the Age column of the blame output ? As it is now, it shows time in either weeks or months.

adam-waldenberg avatar Feb 09 '16 10:02 adam-waldenberg

Hi @adam-waldenberg, both suggestions have been implemented. Please check and let me know if anything else needs to be done. Thanks!

sobomax avatar Feb 09 '16 20:02 sobomax

Thank you. I will play with this. I'm wondering if a -M flag is the best option for this... Maybe removing -w and introducing a "--period" flag that takes a string such as "m" (for months) or "wm" (for weeks and months) would be an idea... It would also make it more flexible. I'm torn, though. I need to think about it :).

adam-waldenberg avatar Feb 10 '16 11:02 adam-waldenberg