summaryrefslogtreecommitdiff
path: root/jamos
diff options
context:
space:
mode:
Diffstat (limited to 'jamos')
-rwxr-xr-xjamos12
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)