Some more type hints?

common-workflow-language · Jul 7, 2017 · 476205b · 476205b
1 parent 21458b8
commit 476205b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/cwltool/builder.py b/cwltool/builder.py
@@ -53,6 +53,7 @@ def __init__(self):  # type: () -> None
         self.job_script_provider = None  # type: Any
 
     def build_job_script(self, commands):
+        # type: (List[str]) -> Text
         build_job_script_method = getattr(self.job_script_provider, "build_job_script", None)  # type: Callable[[Builder, List[str]], Text]
         if build_job_script_method:
             return build_job_script_method(self, commands)

diff --git a/cwltool/job.py b/cwltool/job.py
@@ -192,7 +192,7 @@ def _execute(self, runtime, env, rm_tmpdir=True, move_outputs="move"):
 
             commands = [Text(x).encode('utf-8') for x in runtime + self.command_line]
             job_script_contents = None  # type: Text
-            builder = getattr(self, "builder", None)
+            builder = getattr(self, "builder", None)  # type: Builder
             if builder is not None:
                 job_script_contents = builder.build_job_script(commands)
             rcode = _job_popen(