Comment 5 for bug 79215

Revision history for this message
Daniel Holbach (dholbach) wrote :

Thanks a lot Fernando: awesome work. I committed a modified version to ~dholbach/bughelper/bughelper.dev - please everybody test and check.

Some minor nitpicks:
 - please use an indentation level of 4 spaces (PEP-8 wants that too and we use it throughout)
 - please makes lines only max. 79 chars long
 - I added your name to the contributors list instead of adding it to the header file and also mentioned you in the commit log - I hope that's ok for you
 - and also changed the variable names in a few places to make it more consistent

One change I committed with this change is Sébastien's request to drop the '-a' option. We do '--all' pages by default now.

(The command line options changed in a few places as well - please take care of that in the documentation - Alex Muntada?)

Thanks again Fernando - nice work.