Skip to content
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

Added support for blacklists for authors and paths #47

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Monnoroch
Copy link

fix #25 fix #24

@Monnoroch
Copy link
Author

I actually have tested it, it outputs the same results for default config, and successfully filters all stats (at least in my case).

@Monnoroch Monnoroch force-pushed the master branch 4 times, most recently from 22e5439 to 8374236 Compare March 15, 2015 13:12
@dehru
Copy link

dehru commented Oct 19, 2015

Hey monnoroch, I'm trying to use your fork. I'm getting this error:

Traceback (most recent call last):
  File "/Users/cromed/workspace/forked-gitstats/gitstats/gitstats", line 1511, in <module>
    g.run(sys.argv[1:])
  File "/Users/cromed/workspace/forked-gitstats/gitstats/gitstats", line 1486, in run
    data.collect(gitpath)
  File "/Users/cromed/workspace/forked-gitstats/gitstats/gitstats", line 450, in collect
    time, rev, author = revline.split(' ')
ValueError: too many values to unpack

Any ideas?

@Monnoroch
Copy link
Author

Sorry, but I have no memory of sending this PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants