Commit Graph

228 Commits

Author SHA1 Message Date
Ritiek Malhotra
b968b5d206 Scrape YouTube by default and optionally use YouTube API to perform searches (#250)
* YouTube scraping

* Cleanup GenerateYouTubeURL class

* Some minor improvements

* Add test to fetch title with and without api key
2018-03-09 20:40:15 +05:30
Nitesh Sawant
a8f261edae Changes as per second PR review 2018-02-05 23:01:44 +05:30
Nitesh Sawant
0e3249646f Made Changes as per comments on PR
removed the unnecessary comma and space at the end!
 put the closing bracket on the previous line, as it's done with all the other parser.add_argument calls.
remove the pass - it's completely unneccesary.
2018-02-04 22:36:04 +05:30
Ritiek Malhotra
d77ad8f6e3 Show docker pulls shield 2018-01-29 21:28:58 +05:30
Ritiek Malhotra
0cd85b59bd Code coverage integration (#216)
* Cleaner travis.yml?

* Do we really need wget

* pytest --cov

* Add codecov badge
2018-01-22 20:50:30 +05:30
Ritiek Malhotra
fcfebc55e6 Save file names using a custom format specifiers (#205)
* Use custom formats to generate file name

* Do not mess up search term

* Fix tests

* Fix conflicting names

* Fix subprocess call on file paths with spaces

* Create directories and keep spaces as defaults

* Fix config merge

* Remove underscores from default file format

* Remove debug info

* Show possible formats in usage help
2018-01-19 13:58:27 +05:30
Miguel Piedrafita
1fe94c9896 Add a config option to preserve spaces in file names (#201)
* Add a config option to preserve spaces in file names

* Typo

* Fix replacing in path
2018-01-15 13:41:11 +05:30
Ritiek Malhotra
66e16e4b33 Shell identifier 2018-01-13 17:46:07 +05:30
Ritiek Malhotra
9f35471f3a Read configuration from config.yml (#200)
* Read from config.yml

* Mention config.yml

* Require PyYAML >= 3.12
2018-01-13 16:59:44 +05:30
Ritiek Malhotra
48e323dca5 Update usage help 2018-01-12 22:02:45 +05:30
Ritiek Malhotra
58cfa121f3 Write lyrics to track metadata (#194)
* TODO: Don't throw traceback if no lyrics found

* Add more metadata fields

* Refactor debug logging

* Fix traceback when lyrics not found

* It already vomits metadata :3

* Bump lyricwikia >= 0.1.8
2018-01-11 02:13:23 +05:30
Ritiek Malhotra
cb738ca731 Die typo! 2018-01-07 21:23:32 +05:30
Ritiek Malhotra
2c4e8ac728 Update Docker usage 2018-01-07 21:16:56 +05:30
Ritiek Malhotra
05e5135892 Link to CONTRIBUTING.md 2018-01-07 14:46:37 +05:30
Vishnunarayan K I
4d664956cd Switch to youtube API (#191)
* Switch to youtube API

* Fix test failures

* Minor fix
2018-01-06 18:28:36 +05:30
Ritiek Malhotra
57055eb65d Update usage 2018-01-06 13:08:23 +05:30
Linus Groh
e175608135 Change return code for KeyboardInterrupt 2018-01-04 13:13:15 +01:00
Ritiek Malhotra
3edfb9a5de Mention new --overwrite option 2017-12-31 12:47:00 +05:30
Victor Yap
01bb783724 Add --overwrite option (#182)
* Add overwrite option

* Fix tests

* address changes requested
2017-12-31 12:43:34 +05:30
Linus Groh
64548b6894 Fix link and language in README.md 2017-12-30 04:50:26 +01:00
Ritiek Malhotra
f5587d69b5 Update docker instructions 2017-12-21 16:25:33 +05:30
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
58f54c0041 Add docker build status 2017-12-21 14:13:38 +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
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
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
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 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
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
Linus Groh
3812de3a43 Update README.md to avoid more py2 related issues 2017-09-29 00:23:46 +02:00
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
Ritiek Malhotra
c73c650383 Fix typo in an example command 2017-08-29 23:05:28 +05:30
Linus Groh
ddd57eda88 Fix wrong command line option in README.md 2017-08-25 14:20:32 +02:00
Ritiek Malhotra
dc6019a1a5 Reduce Travis CI build time by more than 90% (#115)
Travis CI now downloads a pre-compiled FFmpeg binary instead of building one on every run. Test duration dropped from 10 mins to about 50 secs.
2017-07-18 14:43:33 +05:30
Ritiek Malhotra
4a2bbb2535 Expect Python to be already in PATH (#114)
* Expect python to already be in PATH

* Expect python to be in PATH

* Fix special handlling of conversion on windows

* Some cleaning

* Update docs
2017-07-17 20:25:11 +05:30
Ritiek Malhotra
9564a71035 Add -p option to directly download playlists with URL (#110)
* Rough implementation of playlist link

* Clean code

* Add information about playlist URL

* Update README.md

* Fix bug where script selects wrong playlist

* Minor improvements

* Minor improvements
2017-07-12 13:37:15 +05:30
Linus
75be7285c0 Add description of the new -f option in README.md; Fix minor issues 2017-07-11 18:48:02 +02:00
Ritiek Malhotra
5add1fd0f8 Drop python 2 compatibility (#107) 2017-07-11 20:32:26 +05:30
Linus Groh
b73d0b1049 Fix rendering issue in README.md 2017-07-11 16:14:17 +02:00