Martin Panter added the comment:

Jacek: If we used the ModuleType check, and somebody adds a module-level 
constant (like logging.CRITICAL = 50), the test will automatically detect if 
they forget to update __all__. That is what I meant by the test being stricter. 
But it looks like you went for the other option, which has its own relative 
advantages :)

Will try to to a proper review later when I get a chance.

----------

_______________________________________
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