From 7161387e6be222486391e75eab2bf06ef58401dd Mon Sep 17 00:00:00 2001 From: Jackson Taylor Date: Tue, 14 Dec 2021 21:58:57 -0500 Subject: Add TODOS Mostly for install and how I want it to work in the future --- jamos | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'jamos') diff --git a/jamos b/jamos index 718a698..9e6c850 100755 --- a/jamos +++ b/jamos @@ -27,6 +27,7 @@ AUDIO_OPTIONS = { 'writeinfojson': True } +# TODO: pass this instead, to make testing easier ytdl = youtube_dl.YoutubeDL(AUDIO_OPTIONS) @@ -61,7 +62,7 @@ def get_all_files(directory): return files -# TODO: Make this better with argparse +# TODO: switch command line args to argparse def get_playlist_url(): return sys.argv[1] @@ -78,7 +79,9 @@ def get_video_urls_in_playlist(playlist_url): def move_file(file, metadata): - # TODO: Move this to its own function to have our own jamos metadata? + # TODO: Pass a jamos specific metadata object/dict + # Then we can do this validation all at once + artist = 'unknownartist' album = 'unknownalbum' title = 'unknownsong' @@ -151,14 +154,15 @@ if __name__ == "__main__": # Get the playlist url from the command line playlist_url = get_playlist_url() + # TODO: Save urls to file so we can start in the + # middle of the playlist if needed urls = get_video_urls_in_playlist(playlist_url) for url in urls: try: - # TODO: make this continue downloading if a song fails download_song(url) except Exception as ex: - # TODO: Handle this a different way + # TODO: Handle this better print(ex) files = get_all_files(MUSIC_DIRECTORY) -- cgit v1.2.3