Add TODOS

Mostly for install and how I want it to work in the future

1 files changed, 8 insertions, 4 deletions

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)