Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add scanfilter to Module constructor #139

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
49 changes: 29 additions & 20 deletions pdoc/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -479,6 +479,23 @@ def __lt__(self, other):
return self.refname < other.refname


def _iter_modules(paths):
"""
Custom implementation of `pkgutil.iter_modules()`
because that one doesn't play well with namespace packages.
See: https://github.com/pypa/setuptools/issues/83
"""
from os.path import isdir, join, splitext
for pth in paths:
for file in os.listdir(pth):
if file.startswith(('.', '__pycache__', '__init__.py')):
continue
if file.endswith(_SOURCE_SUFFIXES):
yield splitext(file)[0]
if isdir(join(pth, file)) and '.' not in file:
yield file


class Module(Doc):
"""
Representation of a module's documentation.
Expand All @@ -491,14 +508,18 @@ class Module(Doc):
__slots__ = ('supermodule', 'doc', '_context', '_is_inheritance_linked')

def __init__(self, module: Union[ModuleType, str], *, docfilter: Callable[[Doc], bool] = None,
supermodule: 'Module' = None, context: Context = None):
scanfilter: Callable[[str], bool] = None, supermodule: 'Module' = None,
context: Context = None):
"""
Creates a `Module` documentation object given the actual
module Python object.

`docfilter` is an optional predicate that controls which
sub-objects are documentated (see also: `pdoc.html()`).

`scanfilter` is an optional predicate that controls which
submodules are ignored during parsing.

`supermodule` is the parent `pdoc.Module` this module is
a submodule of.

Expand Down Expand Up @@ -563,23 +584,7 @@ def is_from_this_module(obj):
# If the module is a package, scan the directory for submodules
if self.is_package:

def iter_modules(paths):
"""
Custom implementation of `pkgutil.iter_modules()`
because that one doesn't play well with namespace packages.
See: https://github.com/pypa/setuptools/issues/83
"""
from os.path import isdir, join, splitext
for pth in paths:
for file in os.listdir(pth):
if file.startswith(('.', '__pycache__', '__init__.py')):
continue
if file.endswith(_SOURCE_SUFFIXES):
yield splitext(file)[0]
if isdir(join(pth, file)) and '.' not in file:
yield file

for root in iter_modules(self.obj.__path__):
for root in _iter_modules(self.obj.__path__):
# Ignore if this module was already doc'd.
if root in self.doc:
continue
Expand All @@ -590,9 +595,13 @@ def iter_modules(paths):

assert self.refname == self.name
fullname = "%s.%s" % (self.name, root)

if scanfilter and not scanfilter(fullname):
continue

self.doc[root] = m = Module(import_module(fullname),
docfilter=docfilter, supermodule=self,
context=self._context)
docfilter=docfilter, scanfilter=scanfilter,
supermodule=self, context=self._context)
# Skip empty namespace packages because they may
# as well be other auxiliary directories
if m.is_namespace and not m.doc:
Expand Down