ID

8eab2a6

date

2024-02-15 21:51:21+00:00

author

Alex Chan <alex@alexwlchan.net>

parent

0d77cb6

message

Don't print info about already-downloaded videos

changed files

1 file, 4 additions, 1 deletion

• web/save_youtube_videos.py

Changed files

web/save_youtube_videos.py (4881) → web/save_youtube_videos.py (4983)

diff --git a/web/save_youtube_videos.py b/web/save_youtube_videos.py
index e74a884..5d1a1fe 100755
--- a/web/save_youtube_videos.py
+++ b/web/save_youtube_videos.py
@@ -10,6 +10,7 @@ import pathlib
 import subprocess
 import sys
 import textwrap
+from typing import Literal
 
 import hyperlink
 from sqlite_utils import Database
@@ -76,7 +77,8 @@ def log_result(format_template):
                 print(termcolor.colored(f"✘ {description}\n{wrapped_error}", "red"))
                 raise
             else:
-                print(termcolor.colored(f"✔ {description}", "green"))
+                if result == 'downloaded':
+                    print(termcolor.colored(f"✔ {description}", "green"))
                 return result
 
         return wrapper
@@ -164,6 +166,7 @@ def download_video(*, video_id, download_root):
     try:
         youtube_dl(*cmd, cwd=download_dir)
         print(download_dir)
+        return 'downloaded'
     except subprocess.CalledProcessError as err:  # pragma: no cover
         print(f"Unable to download {video_url}: {err}", file=sys.stderr)
         raise