refactor: use list comprehension rather filter+lambda

jupyterhub · Dec 13, 2023 · cc70092 · cc70092
1 parent 6fd2ad2
commit cc70092
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/repo2docker/buildpacks/base.py b/repo2docker/buildpacks/base.py
@@ -612,16 +612,18 @@ def _filter_tar(tarinfo):
 
         exclude = []
 
-        for ignore_file in [".dockerignore", ".containerignore"]:
-            if os.path.exists(ignore_file):
-                with open(ignore_file, "r") as f:
+        for ignore_file_name in [".dockerignore", ".containerignore"]:
+            if os.path.exists(ignore_file_name):
+                with open(ignore_file_name, "r") as ignore_file:
+                    cleaned_lines = [
+                        line.strip() for line in ignore_file.read().splitlines()
+                    ]
                     exclude.extend(
-                        list(
-                            filter(
-                                lambda x: x != "" and x[0] != "#",
-                                [l.strip() for l in f.read().splitlines()],
-                            )
-                        )
+                        [
+                            line
+                            for line in cleaned_lines
+                            if line != "" and line[0] != "#"
+                        ]
                     )
 
         if not exclude: