Commit Graph

674 Commits

Author SHA1 Message Date
Ritiek Malhotra c75a243690 Improve formatting 2017-12-21 15:44:49 +05:30
Ritiek Malhotra fddfeb43cd Add docker automated shield and other changes 2017-12-21 15:43:15 +05:30
Ritiek Malhotra bbfc3e21c2 Update Dockerfile 2017-12-21 15:06:56 +05:30
Ritiek Malhotra 58f54c0041 Add docker build status 2017-12-21 14:13:38 +05:30
Ritiek Malhotra cc018ea0a2 Update Dockerfile 2017-12-21 14:00:09 +05:30
Ritiek Malhotra 6242c2d7c0 Update Dockerfile 2017-12-21 13:46:27 +05:30
Ritiek Malhotra 93388418f0 Show filepath in debug mode 2017-12-21 13:24:36 +05:30
Ritiek Malhotra 0a27e1d523 Merge pull request #169 from bcardiff/docker
Add docker image
2017-12-16 11:01:42 +05:30
Brian J. Cardiff 5811403116 Add docker image 2017-12-15 13:12:00 -03:00
ritiek fe11ade687 Log name if song already exists 2017-12-15 20:10:57 +05:30
Ritiek Malhotra df513acc35 Add logging capability (#175)
* Refactoring and addition of logzero (#172)

* Refactored convert.py and added logging.

* Added logging and refactored.

* Added logzero to requirements.txt

* Added logging to metadata.py

* Created a log in misc.py. Updated slugify import.

* Some general improvement

* Improve message layout

* Improve test mechanism

* Implement debug level logging

* Fix some minor mistakes

* Make pytest happy

* Remove unimplemented --verbose option

* Update ISSUE_TEMPLATE.md

* Rename LICENSE

* Remove obvious from log.debug()

* Show track URL when writing to file (debug)
2017-12-15 19:57:57 +05:30
Ritiek Malhotra 8ea89f9d1c Import sys to deal with sys.exit() 2017-12-09 00:36:30 +05:30
Aareon Sullivan 77baa71d24 Minor tweaks (#154)
* Minor tweaks

Use at your own risk. Just thought I’d contribute at least a little bit

* Exit code 0

Exit code 0 is standard for notifying the system of a regular, non-error exit.

* No longer need unnecessary flag

* Fix `already_tagged` not defined

* <3 Zen of Python

* Silly me
2017-12-08 15:17:36 +05:30
ritiek 84b6ec601b Remove debug code 2017-12-05 22:29:20 +05:30
ritiek 9bee8d8787 Fix parameters to write_playlist() with -u option 2017-12-05 22:28:21 +05:30
Brian J. Cardiff c8d71954b4 Fix typo in README (#168) 2017-12-03 11:53:06 +05:30
Ritiek Malhotra 6075a829e5 Return codes depicting success/failure (#164)
* Add error codes for fetch playlist failures

* Add return codes to README.md
2017-11-30 10:44:46 +05:30
ritiek d39272d6a2 Avoid use of os.chdir() 2017-11-30 10:20:36 +05:30
fanlp3 6ac60000e6 fixes: #159 Refactored grab_playlist (#160)
* Refactored grab_playlist

grab_playlist was looping through the user's playlist until it finds the playlist.
In some cases a public playlist isn't in the user's playlists
Using a more direct approach, querying for the tracks of a playlist

* Remove old `def write_playlist()`

* Playlist URL download now works on private playlists
2017-11-30 09:49:38 +05:30
Ritiek Malhotra 7d7f4b75f7 Fix link in README 2017-11-25 19:52:01 +05:30
Ritiek Malhotra 24614795af Remove outdated usage image 2017-11-19 18:48:50 +05:30
ritiek b4619d0621 Fix tests 2017-11-14 00:00:08 +05:30
ritiek ade74e8272 Call generate_metadata() only once 2017-11-13 23:45:59 +05:30
ritiek 618476eca7 Update ISRC metadata 2017-11-13 21:33:29 +05:30
Linus Groh b08213c8b0 Fix example usage description in README.md 2017-11-12 01:15:32 +01:00
Ritiek Malhotra 7f4cb749c8 Add note for collaborative playlists 2017-11-11 19:26:08 +05:30
ritiek 0deea6b384 Fix error when playlist not found 2017-11-11 19:17:10 +05:30
Ritiek Malhotra fd130f8626 Error out if playlist not found 2017-11-11 19:13:37 +05:30
Ritiek Malhotra b879f10401 Update all example commands to NCS music 2017-11-10 21:42:05 +05:30
Ritiek Malhotra 5468730ad3 Update contributing 2017-11-10 21:28:05 +05:30
Ritiek Malhotra cfd392c6ce Fix access token regeneration 2017-10-28 01:07:50 +05:30
Ritiek Malhotra 2780ba405f Catch only IndexError 2017-10-27 14:37:26 +05:30
ritiek 2f0018adce Fix ValueError if publisher not found 2017-10-20 14:27:37 +05:30
Linus Groh a3cfcfcd81 Add note about spaces in filenames in README.md 2017-10-17 22:59:07 +02:00
Valérian Galliat 237b4cca7e Add an option to grab an album (#142) 2017-10-08 19:43:55 +05:30
Ritiek Malhotra 98d21bbad9 Fix NoneType not scriptable if song not on Spotify (#140) 2017-09-29 12:38:15 +05:30
Linus Groh 3812de3a43 Update README.md to avoid more py2 related issues 2017-09-29 00:23:46 +02:00
ritiek 9068f6008e Fix albumart exception in .M4A downloads 2017-09-27 10:26:03 +05:30
ritiek f9e85d3796 albumart out of range exception 2017-09-27 10:22:25 +05:30
Ritiek Malhotra 97a8c21eb9 Download songs using YouTube URL (#135)
* Download from YouTubr URL

* Slugify title only for YouTube URL's
2017-09-27 09:56:47 +05:30
Linus Groh 17e6d1fa2e Update README.md to include a note about pip2/3 2017-09-18 00:03:25 +02:00
Ritiek Malhotra 31b91bd6da Use Python 3.6+ on Windows to avoid Unicode errors 2017-09-05 21:41:22 +05:30
Pierre Gérard 633fb9b1bd [fix] add wait to avoid infinite retry looping (#126) 2017-08-30 20:01:23 +05:30
Ritiek Malhotra c73c650383 Fix typo in an example command 2017-08-29 23:05:28 +05:30
Linus Groh 6b3dafa99b Merge pull request #121 from lkgarrison/master
Improves accuracy of selected youtube video
2017-08-26 17:28:38 +02:00
Linus Groh ddd57eda88 Fix wrong command line option in README.md 2017-08-25 14:20:32 +02:00
ritiek 0178307b20 Use tinydownload to download ffmpeg binary 2017-08-12 19:33:38 +05:30
Luke Garrison 9d25197a5f Improves accuracy of selected youtube video
Finds a balance between viewcount (Youtube's relevancy) and proper song
duration based on Spotify duration. Thus, a Youtube video that is 30
seconds longer than the Spotify track will not be considered

Takes the first result from Youtube (using Youtube's original ordering
based on relevance) that has a similar duration to the Spotify song

Fixes a bug where if there were no suitable videos for a song, the
program would infinitely loop. The program will now retry to find a song
up to 5 times before moving on (this is necessary because occasionally the
song isn't properly fetched or parsed from Youtube)

Fixes bug where songs that are retried were appended to the playlist
file without being separated by a newline
2017-08-09 21:42:31 -04:00
Ritiek Malhotra 7e0fdfbce3 Update .travis.yml 2017-08-03 11:01:10 +00:00
Ritiek Malhotra ed630823b0 Update .travis.yml 2017-08-03 10:51:14 +00:00