Fix broken type hints for nested functions

Author: Pebaz

nimporter/lib.py

@@ -218,7 +218,7 @@ def ensure_nimpy() -> None:
     ic()
 
     try:
-        run_process('nimble')
+        run_process(['nimble'])
     except FileNotFoundError as e:
         error = CompilationFailedException('Nim not installed or not on path')
         raise error from e

nimporter/nexporter.py

@@ -1,4 +1,5 @@
 import os
+import re
 import sys
 from typing import *
 from pathlib import Path
@@ -257,27 +258,6 @@ def compile_extensions_to_c(platforms: List[str], root: Path) -> None:
     return
 
 
-def _is_valid_identifier(string: str) -> Union[Match[str], None, bool]:
-        import re
-        match = re.search('^[A-Za-z_][A-Z-a-z0-9_\\-]*', string)
-        return match and len(match.string) == len(string)
-
-
-def _is_semver(string: str) -> bool:
-    try:
-        lib_name, lib_version = string.rsplit('-', maxsplit=1)
-        assert _is_valid_identifier(lib_name)
-
-        major, minor, patch = lib_version.split('.')
-        assert major.isdigit()
-        assert minor.isdigit()
-        assert patch.isdigit()
-
-        return True
-    except:
-        return False
-
-
 def prevent_win32_max_path_length_error(path: Path) -> None:
     """
     Nim generates C files that contain `@` symbols to encode the original path
@@ -296,6 +276,24 @@ def prevent_win32_max_path_length_error(path: Path) -> None:
     That's a lot less characters!
     """
 
+    def _is_valid_identifier(string: str) -> Union[re.Match[str], None, bool]:
+        match = re.search('^[A-Za-z_][A-Z-a-z0-9_\\-]*', string)
+        return match and len(match.string) == len(string)
+
+    def _is_semver(string: str) -> bool:
+        try:
+            lib_name, lib_version = string.rsplit('-', maxsplit=1)
+            assert _is_valid_identifier(lib_name)
+
+            major, minor, patch = lib_version.split('.')
+            assert major.isdigit()
+            assert minor.isdigit()
+            assert patch.isdigit()
+
+            return True
+        except:
+            return False
+
     for item in path.iterdir():
         if item.is_file() and item.name.startswith('@m'):