coverage: Don't expose function as public

We don't want this to be a public API.

Signed-off-by: Stephen Finucane <[email protected]>

Author: stephenfin

sphinx/ext/coverage.py

@@ -66,7 +66,7 @@ def _add_row(col_widths: list[int], columns: list[str], separator: str) -> Itera
     yield _add_line(col_widths, separator)
 
 
-def load_modules(mod_name: str, ignored_module_exps: list[re.Pattern[str]]) -> set[str]:
+def _load_modules(mod_name: str, ignored_module_exps: list[re.Pattern[str]]) -> set[str]:
     """Recursively load all submodules.
 
     :param mod_name: The name of a module to load submodules for.
@@ -93,7 +93,7 @@ def load_modules(mod_name: str, ignored_module_exps: list[re.Pattern[str]]) -> s
             continue
 
         if sub_mod_info.ispkg:
-            modules |= load_modules(f'{mod_name}.{sub_mod_info.name}', ignored_module_exps)
+            modules |= _load_modules(f'{mod_name}.{sub_mod_info.name}', ignored_module_exps)
         else:
             for exp in ignored_module_exps:
                 if exp.match(sub_mod_info.name):
@@ -229,7 +229,7 @@ def build_py_coverage(self) -> None:
             modules = set()
             for mod_name in self.module_names:
                 try:
-                    modules |= load_modules(mod_name, self.mod_ignorexps)
+                    modules |= _load_modules(mod_name, self.mod_ignorexps)
                 except ImportError as err:
                     # TODO(stephenfin): Define a subtype for all logs in this module
                     logger.warning(__('module %s could not be imported: %s'), mod_name, err)