clean up of types and names

Author: aaronbog

src/pythainer/builders/__init__.py

@@ -222,15 +222,15 @@ def add_packages(self, packages: List[str]) -> None:
         """
         self._build_commands.append(AddPkgDockerBuildCommand(packages=packages))
 
-    def copy(self, file_path: PathType, destination_path: PathType) -> None:
+    def copy(self, source_path: Path, destination_path: Path) -> None:
         """
         Copies a file to the docker container
 
         Parameters:
-            filename (PathType): The file to copy to the container.
-            destination (PathType): The location to place the file within the Docker container.
+            source_path (Path): The file or folder to copy to the container.
+            destination_path (Path): The location to place the file or folder within the Docker container.
         """
-        self._build_commands.append(CopyDockerBuildCommand(file_path,destination_path))
+        self._build_commands.append(CopyDockerBuildCommand(source_path,destination_path))
 
 
 class DockerBuilder(PartialDockerBuilder):

src/pythainer/builders/cmds.py

@@ -88,6 +88,7 @@ def __init__(self, source_path:Path ,destination_path:Path) -> None:
         self._source_path = source_path
         self._destination_path = destination_path
 
+    # pylint: disable=arguments-differ
     def get_str_for_dockerfile(
         self,
         docker_file_Path: Path,
@@ -101,8 +102,6 @@ def get_str_for_dockerfile(
             str: The command string.
         """
 
-        # TODO: is mkdir needed and if so where?
-
         if os.path.isfile(self._source_path):
             shutil.copyfile(self._source_path, docker_file_Path / self._source_path)
         elif os.path.isdir(self._source_path):