be more explicit in 'common_funcs' usage

Author: sommersoft

adabot/lib/circuitpython_library_validators.py

@@ -27,7 +27,7 @@
 from adabot import github_requests as github
 from adabot import travis_requests as travis
 from adabot import pypi_requests as pypi
-from adabot.lib.common_funcs import *
+from adabot.lib import common_funcs
 
 
 # Define constants for error strings to make checking against them more robust:
@@ -169,7 +169,7 @@ def validate_repo_state(self, repo):
             errors.append(ERROR_MISSING_LICENSE)
         if not repo["permissions"]["push"]:
             errors.append(ERROR_MISSING_LIBRARIANS)
-        if not is_repo_in_bundle(full_repo["clone_url"], self.bundle_submodules) and \
+        if not common_funcs.is_repo_in_bundle(full_repo["clone_url"], self.bundle_submodules) and \
            not repo["name"] in BUNDLE_IGNORE_LIST:  # Don't assume the bundle will
                                                     # bundle itself and possibly
                                                     # other repos.
@@ -588,9 +588,9 @@ def validate_readthedocs(self, repo):
                 return [ERROR_RTD_SUBPROJECT_FAILED]
             rtd_subprojects = {}
             for subproject in rtd_response.json()["subprojects"]:
-                rtd_subprojects[sanitize_url(subproject["repo"])] = subproject
+                rtd_subprojects[common_funcs.sanitize_url(subproject["repo"])] = subproject
 
-        repo_url = sanitize_url(repo["clone_url"])
+        repo_url = common_funcs.sanitize_url(repo["clone_url"])
         if repo_url not in rtd_subprojects:
             return [ERROR_RTD_SUBPROJECT_MISSING]
 
@@ -789,6 +789,6 @@ def validate_in_pypi(self, repo):
         if not (repo["owner"]["login"] == "adafruit" and
                 repo["name"].startswith("Adafruit_CircuitPython")):
             return []
-        if not repo_is_on_pypi(repo):
+        if not common_funcs.repo_is_on_pypi(repo):
             return [ERROR_NOT_ON_PYPI]
         return []