Jacek Kołodziej added the comment:

Hi! This is my first attempt at contributing so as always, feedback will be 
well appreciated. :)

I meant to start small so I took a shot with csv module. In test, initial 
expected set contains QUOTE_* because they don't provide __module__ attribute, 
and __doc/version__ because they are already in csv.__all__ (should they?).

I've made a few PEP8-related fixes just around code I've touched (so they 
aren't completely unrelated). Is that ok?

----------
nosy: +Unit03
Added file: http://bugs.python.org/file39716/Issue23883_csv_all.patch

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue23883>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to