Merge pull request #53 from DavidCEllis/fix-unusable-python

Check if install_details returned a value before yielding

Author: DavidCEllis

src/ducktools/pythonfinder/win32/pyenv_search.py

@@ -72,4 +72,6 @@ def get_pyenv_pythons(
                 executable = os.path.join(p.path, "python.exe")
 
             if os.path.exists(executable):
-                yield finder.get_install_details(executable, managed_by="pyenv")
+                install = finder.get_install_details(executable, managed_by="pyenv")
+                if install:
+                    yield install

src/ducktools/pythonfinder/win32/registry_search.py

@@ -113,11 +113,13 @@ def get_registered_pythons(finder: DetailFinder | None = None) -> Iterator[Pytho
                             if python_path:
                                 # Pyenv puts architecture information in the Version value for some reason
                                 if os.path.isfile(python_path):
-                                    yield finder.get_install_details(
+                                    details = finder.get_install_details(
                                         python_path,
                                         managed_by=metadata["Company"],
                                         metadata=metadata,
                                     )
+                                    if details:
+                                        yield details
 
             finally:
                 if base_key: