Today I came across the following pylint error:
invalid-all-object (E0604):
Invalid object %r in __all__, must contain only strings Used when an invalid (non-string) object occurs in __all__.
And I'm quite curious to why is it considered incorrect to expose objects directly?
Because it's supposed to be a list of names, not values:
If the list of identifiers is replaced by a star (
'*'
), all public names defined in the module are bound in the local namespace for the scope where theimport
statement occurs.The public names defined by a module are determined by checking the module’s namespace for a variable named
__all__
; if defined, it must be a sequence of strings which are names defined or imported by that module. The names given in__all__
are all considered public and are required to exist. If__all__
is not defined, the set of public names includes all names found in the module’s namespace which do not begin with an underscore character ('_'
).__all__
should contain the entire public API. It is intended to avoid accidentally exporting items that are not part of the API (such as library modules which were imported and used within the module). [Language Reference]