search

dysosmus / gitinspector

Automatically exported from code.google.com/p/gitinspector
GNU General Public License v3.0
0 stars 0 forks source link

Feature request: Exclude authors. #5

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
I'd be great to have an option to exclude some authors from statistics... 
especially when you want to grade a student project that is based on some 
existing repository, and you only want to see the commits of a specified group 
of people.

Unfortunately I can't implement that myself... but maybe someone else will also 
be interested, and will do necessary changes.

Original issue reported on code.google.com by wb...@jitsolutions.pl on 19 Jun 2013 at 11:46

GoogleCodeExporter commented 9 years ago 
I like the idea and I think it would be a great feature to add.

Gitinspector already has the -x/--exclude flag for excluding files that match a 
certain string/name (it can be specified multiple times). How about expanding 
that so one could do something like this:

gitinspector.py -x="author:John Smith" -x "author:Robert Brown"

This way; -x could also be expanded in the future to include other types of 
exclusion patterns.

However, implementing it that way does bring up a conundrum; What happens with 
very big repositories that have many (for example 100) authors? Specifying it 
some ~100 times isn't my idea of a good time. Maybe it's time for a 
-i/--include flag also?  My thinking is that -i/-x could be specified multiple 
times in different variations to create different matching rules.

In that case it would be a matter of doing something like this:

gitinspector.py -x "author:*" -i "author:James Johnsson" -i "author:Michael 
Davids"

The idea is that the above would exclude all authors but include "James 
Johnsson" and "Michael Davids" in the statistics.

Maybe -x "author:*" should be the default behaviour if the first pattern 
matching rule is a -i "author:...", otherwise nothing would happen.

Tell me what you think and if you have any better ideas.

/Adam Waldenberg

Original comment by gitinspe...@ejwa.se on 20 Jun 2013 at 1:22

GoogleCodeExporter commented 9 years ago 
In any case, I will look into this eventually. However, I will put it on hold 
until some time after the 0.3.0 release.

/Adam Waldenberg

Original comment by gitinspe...@ejwa.se on 30 Jun 2013 at 4:43

GoogleCodeExporter commented 9 years ago

Original comment by gitinspe...@ejwa.se on 30 Jun 2013 at 6:44

GoogleCodeExporter commented 9 years ago

Original comment by gitinspe...@ejwa.se on 20 Jul 2013 at 8:31

GoogleCodeExporter commented 9 years ago

Original comment by gitinspe...@ejwa.se on 29 Jul 2013 at 10:46

GoogleCodeExporter commented 9 years ago 
This issue was closed by revision 1edae66fee4f.

Original comment by gitinspe...@ejwa.se on 30 Jul 2013 at 4:34

GoogleCodeExporter commented 9 years ago 
Done. Please repport any problems you may encounter. For documentation on how 
to use this; see the revision referenced above

/Adam Waldenberg

Original comment by gitinspe...@ejwa.se on 30 Jul 2013 at 4:38

GoogleCodeExporter commented 9 years ago

Original comment by gitinspe...@ejwa.se on 13 Jan 2014 at 11:00