remove bad to_list() call, add missing doc

Author: rickeylev

python/private/py_library.bzl

@@ -234,10 +234,10 @@ def _get_imports_and_venv_symlinks(ctx, semantics):
     if VenvsSitePackages.is_enabled(ctx):
         package, version_str = _get_package_and_version(ctx)
         imports = ctx.attr.imports
-        if len(imports) == 0:
+        if len(imports.to_list()) == 0:
             fail("When venvs_site_packages is enabled, exactly one `imports` " +
                  "value must be specified, got 0")
-        elif len(imports) > 1:
+        elif len(imports.to_list()) > 1:
             fail("When venvs_site_packages is enabled, exactly one `imports` " +
                  "value must be specified, got {}".format(imports))
 

python/private/venv_runfiles.bzl

@@ -198,6 +198,9 @@ def get_venv_symlinks(ctx, files, package, version_str, site_packages_root):
 
     Args:
         ctx: {type}`ctx` the current ctx.
+        files: {type}`list[File]` the underlying files that are under
+            `site_packages_root` and intended to be part of the venv
+            contents.
         package: {type}`str` the Python distribution name.
         version_str: {type}`str` the distribution's version.
         site_packages_root: {type}`str` prefix under which files are