add back in Python 3.7 for one last release

Author: mr-c

.github/workflows/ci-tests.yml

@@ -32,8 +32,18 @@ jobs:
     strategy:
       matrix:
         py-ver-major: [3]
-        py-ver-minor: [8, 9, 10, 11, 12]
+        py-ver-minor: [7, 8, 9, 10, 11, 12]
         step: [lint, unit, bandit, mypy]
+        exclude:
+          - py-ver-major: 3
+            py-ver-minor: 7
+            step: mypy
+          - py-ver-major: 3
+            py-ver-minor: 7
+            step: lint
+          - py-ver-major: 3
+            py-ver-minor: 7
+            step: bandit
 
     env:
       py-semver: ${{ format('{0}.{1}', matrix.py-ver-major, matrix.py-ver-minor) }}

cwltool/checker.py

@@ -5,14 +5,14 @@
     Dict,
     Iterator,
     List,
-    Literal,
     MutableMapping,
     MutableSequence,
     Optional,
     Sized,
     Union,
     cast,
 )
+from typing_extensions import Literal
 
 from schema_salad.exceptions import ValidationException
 from schema_salad.sourceline import SourceLine, bullets, strip_dup_lineno
@@ -509,7 +509,8 @@ def get_step_id(field_id: str) -> str:
 
 
 def is_conditional_step(param_to_step: Dict[str, CWLObjectType], parm_id: str) -> bool:
-    if (source_step := param_to_step.get(parm_id)) is not None:
+    source_step = param_to_step.get(parm_id)
+    if source_step is not None:
         if source_step.get("when") is not None:
             return True
     return False

cwltool/context.py

@@ -12,12 +12,12 @@
     Dict,
     Iterable,
     List,
-    Literal,
     Optional,
     TextIO,
     Tuple,
     Union,
 )
+from typing_extensions import Literal
 
 from ruamel.yaml.comments import CommentedMap
 from schema_salad.avro.schema import Names

cwltool/cwlprov/__init__.py

@@ -6,7 +6,8 @@
 import re
 import uuid
 from getpass import getuser
-from typing import IO, Any, Callable, Dict, List, Optional, Tuple, TypedDict, Union
+from typing import IO, Any, Callable, Dict, List, Optional, Tuple, Union
+from typing_extensions import TypedDict
 
 
 def _whoami() -> Tuple[str, str]:

cwltool/software_requirements.py

@@ -91,8 +91,9 @@ def build_job_script(self, builder: "Builder", command: List[str]) -> str:
             resolution_config_dict=resolution_config_dict,
             conf_file=self.dependency_resolvers_config_file,
         )
-        handle_dependencies: str = ""
-        if dependencies := get_dependencies(builder):
+        dependencies = get_dependencies(builder)
+        handle_dependencies = ""  # str
+        if dependencies:
             handle_dependencies = "\n".join(
                 tool_dependency_manager.dependency_shell_commands(
                     dependencies, job_directory=builder.tmpdir

cwltool/utils.py

@@ -1,7 +1,6 @@
 """Shared functions and other definitions."""
 import collections
 import fcntl
-import importlib.metadata
 import os
 import random
 import shutil
@@ -28,17 +27,16 @@
     Generator,
     Iterable,
     List,
-    Literal,
     MutableMapping,
     MutableSequence,
     NamedTuple,
     Optional,
     Set,
     Tuple,
-    TypedDict,
     Union,
     cast,
 )
+from typing_extensions import Literal, TypedDict
 
 import requests
 from cachecontrol import CacheControl
@@ -47,6 +45,11 @@
 from schema_salad.exceptions import ValidationException
 from schema_salad.ref_resolver import Loader
 
+if sys.version_info >= (3, 8):
+    import importlib.metadata as importlib_metadata
+else:
+    import importlib_metadata
+
 if TYPE_CHECKING:
     from .command_line_tool import CallbackJob, ExpressionJob
     from .job import CommandLineJob, JobBase
@@ -117,7 +120,8 @@ class WorkflowStateItem(NamedTuple):
 
 def versionstring() -> str:
     """Version of CWLtool used to execute the workflow."""
-    if pkg := importlib.metadata.version("cwltool"):
+    pkg = importlib_metadata.version("cwltool")
+    if pkg:
         return f"{sys.argv[0]} {pkg}"
     return "{} {}".format(sys.argv[0], "unknown version")
 

docs/conf.py

@@ -14,7 +14,6 @@
 import sys
 from datetime import datetime
 import time
-import importlib.metadata
 
 sys.path.insert(0, os.path.abspath(".."))
 
@@ -82,7 +81,12 @@
 # so a file named "default.css" will overwrite the builtin "default.css".
 html_static_path = ["_static"]
 
-release = importlib.metadata.version("cwltool")
+
+if sys.version_info >= (3, 8):
+    import importlib.metadata as importlib_metadata
+else:
+    import importlib_metadata
+release = importlib_metadata.version("cwltool")
 version = ".".join(release.split(".")[:2])
 
 autoapi_dirs = ["../cwltool"]

gittaggers.py

@@ -2,13 +2,16 @@
 import sys
 import time
 
-import importlib.metadata
+if sys.version_info >= (3, 8):
+    import importlib.metadata as importlib_metadata
+else:
+    import importlib_metadata
 
 from typing import Any
 
 from setuptools.command.egg_info import egg_info
 
-SETUPTOOLS_VER = importlib.metadata.version("setuptools").split(".")
+SETUPTOOLS_VER = importlib_metadata.version("setuptools").split(".")
 
 RECENT_SETUPTOOLS = (
     int(SETUPTOOLS_VER[0]) > 40

pyproject.toml

@@ -15,4 +15,4 @@ build-backend = "setuptools.build_meta"
 
 [tool.black]
 line-length = 100
-target-version = [ "py38" ]
+target-version = [ "py37" ]

requirements.txt

@@ -7,7 +7,9 @@ schema-salad>=8.4.20230426093816,<9
 prov==1.5.1
 mypy-extensions
 psutil>=5.6.6
+typing-extensions
 importlib_resources>=1.4  # equivalent to Python 3.9
+importlib_metadata;python_version<'3.8'  # equivalent to Python 3.9
 coloredlogs
 pydot>=1.4.1
 argcomplete>=1.12.0