dmotitsk / coursera

Coursera materials downloader.
58 stars 20 forks source link

rename files, use only generic course names, add downloaddirectory configuration #2

Closed skogler closed 12 years ago

dmotitsk commented 12 years ago

Hello, Samuel.

Thank you for your work, this is what I've wanted to add to this script when I had enough time. I have a couple of wishes to your patch. Could you please fix indentation (no more 79 symbols in a line) to be PEP8 compatible. Also I'm not sure if the option of target directory should be in config. Initially I planed to launch this script from the target directory. May be it is a better idea to add a command line option for it. Could you please change a 'class_name' class variable to 'course_name'. I think it will be closer to its purpose. And I think that the default verbose value should be zero. You can always change it with the help of command line argument.

Regards, Dmitry.